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

Sphinx - RSS #1934

Merged
merged 5 commits into from
Jun 15, 2021
Merged

Sphinx - RSS #1934

merged 5 commits into from
Jun 15, 2021

Conversation

AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Apr 20, 2021

See #2, #1385 for context.

This is the RSS generation part, building on PR #1930. It contains the logic for generating RSS

@AA-Turner AA-Turner requested a review from a team as a code owner April 20, 2021 22:44
@hugovk hugovk mentioned this pull request Apr 21, 2021
Makefile Show resolved Hide resolved
pep_rss_gen.py Show resolved Hide resolved
This was referenced May 5, 2021
@AA-Turner AA-Turner force-pushed the sphinx-rss branch 2 times, most recently from a82ddc8 to c46ce31 Compare May 7, 2021 14:56
@pablogsal
Copy link
Member

@AA-Turner This needs rebasing or closing?

@AA-Turner
Copy link
Member Author

Rebased

@pablogsal
Copy link
Member

@AA-Turner What's the best way to test this?

@AA-Turner
Copy link
Member Author

make pep_rss or python pep_rss_gen.py & looking at the RSS file.

@AA-Turner
Copy link
Member Author

AA-Turner commented Jun 12, 2021

Just found https://validator.w3.org/feed/#validate_by_input, seems that the spec only supports one author entry per item, will ammend.

(edit: for future ref, https://stackoverflow.com/questions/5855993/multiple-authors-in-rss-or-atoms)

@AA-Turner
Copy link
Member Author

@pablogsal fixed author issues (W3C validator not entirely happy as it expects only one author, but the spec is silent on this and the SO answer above seemed to confirm that there is no best way to deal with this).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants