-
Notifications
You must be signed in to change notification settings - Fork 1.6k
feat: add feeds for the blog and vulnerabilities #1867
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Deploy Preview for expressjscom-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, everything looks good, @rowanmanning thanks for working on this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is great! Thanks!
This adds two new feeds to the site: `feed.xml` includes all blog posts, and `vulnerabilities.xml` includes all posts tagged with `vulnerabilities`. Resolves expressjs#1763
Jekyll expects tags to be space-delimited, using commas results in the tags themselves containing a comma. See docs: https://jekyllrb.com/docs/posts/#tags Fixing this allows us to stop working around the trailing commas.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@rowanmanning Thanks! |
This adds two new feeds to the site:
feed.xml
includes all blog posts, andvulnerabilities.xml
includes all posts tagged withvulnerabilities
.You can validate the feeds via the W3C Feed Validation Service. I also tried subscribing via Feedbin and all the posts appear as expected.
Resolves #1763