Skip to content
This repository was archived by the owner on Nov 30, 2024. It is now read-only.

Conversation

@dieyushi
Copy link

@dieyushi dieyushi commented May 7, 2013

update those two file to match RSS 2.0 Specification and RFC 4287.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's better to give an option in _config.yml, like named rss_limit.

@dieyushi
Copy link
Author

dieyushi commented Jun 4, 2013

@Lax yeah, it's better.

@fujii fujii mentioned this pull request May 25, 2014
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm using the liquid template date _to_rfc822 (http://jekyllrb.com/docs/templates/)[http://jekyllrb.com/docs/templates/]:

               <pubDate>{{ post.date | date_to_rfc822 }}</pubDate>

Source in #215

@groundh0g groundh0g added this to the v 0.5.0 milestone Apr 4, 2015
@groundh0g groundh0g self-assigned this Apr 4, 2015
@groundh0g
Copy link
Collaborator

Unfortunately, it's been too long to merge this PR as-is. I've made the edits from #157 and #215 locally and am testing tonight. Hope to commit and close soon. Thanks for your contribution. Sorry for the delay in accepting it.

@groundh0g
Copy link
Collaborator

Testing is complete. Works as expected. Thanks for the patch! Will merge the changes to master soonish.

groundh0g added a commit to groundh0g/jekyll-bootstrap that referenced this pull request May 23, 2015
. In a nutshell: limit RSS feeds to 20, correct some bathing issues, format timestamps, shorten TTL.
@groundh0g
Copy link
Collaborator

Incorporated into #272.

@groundh0g groundh0g closed this May 23, 2015
groundh0g added a commit that referenced this pull request May 24, 2015
groundh0g added a commit to groundh0g/jekyll-bootstrap that referenced this pull request May 25, 2015
aterai pushed a commit to aterai/jekyll-bootstrap that referenced this pull request Mar 11, 2019
. In a nutshell: limit RSS feeds to 20, correct some bathing issues, format timestamps, shorten TTL.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants