Skip to content
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

docs: add RSS feed to blog #367

Merged
merged 3 commits into from
May 29, 2024
Merged

docs: add RSS feed to blog #367

merged 3 commits into from
May 29, 2024

Conversation

jrycw
Copy link
Contributor

@jrycw jrycw commented May 27, 2024

This PR adds an RSS feed to the blog, following the instructions from quarto, and addresses the request in #115.

@codecov-commenter
Copy link

codecov-commenter commented May 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.90%. Comparing base (e3649d2) to head (f145900).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #367   +/-   ##
=======================================
  Coverage   85.90%   85.90%           
=======================================
  Files          41       41           
  Lines        4328     4328           
=======================================
  Hits         3718     3718           
  Misses        610      610           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@machow
Copy link
Collaborator

machow commented May 29, 2024

Here's a link to a docs preview: https://docs-preview-rss--gt-python.netlify.app/

@rich-iannone do you mind checking if this looks okay / merging? (I'm guessing it should be good to go?)

afaict it produces this: https://docs-preview-rss--gt-python.netlify.app/blog/index.xml

@rich-iannone rich-iannone self-requested a review May 29, 2024 21:15
Copy link
Member

@rich-iannone rich-iannone left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks @jrycw :)

@rich-iannone rich-iannone merged commit 797ad13 into posit-dev:main May 29, 2024
12 checks passed
@jrycw jrycw deleted the fix-115 branch May 29, 2024 22:22
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants