Skip to content

Infra: Make PEP abstract extration more robust #4356

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.

Sign up for GitHub

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

Merged
AA-Turner merged 3 commits into from
Apr 11, 2025
Merged

Infra: Make PEP abstract extration more robust #4356

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
5c7c23f
generate_rss: make pep_abstract more robust
woodruffw Apr 8, 2025
1def89a
Merge branch 'main' into ww/fix-pep-abstract
woodruffw Apr 8, 2025
5836a73
apply patch from @AA-Turner
woodruffw Apr 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions pep_sphinx_extensions/generate_rss.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,10 @@
continue

if title_node.astext() == "Abstract":
return node.next_node(nodes.paragraph).astext().strip().replace("\n", " ")
elif title_node.astext() == "Introduction":
if (para_node := node.next_node(nodes.paragraph)) is not None:
return para_node.astext().strip().replace("\n", " ")
return ""
if title_node.astext() == "Introduction":

Check warning on line 66 in pep_sphinx_extensions/generate_rss.py

View check run for this annotation

Codecov / codecov/patch

pep_sphinx_extensions/generate_rss.py#L63-L66 

Added lines #L63 - L66 were not covered by tests
introduction = node.next_node(nodes.paragraph).astext().strip().replace("\n", " ")

return introduction
Expand Down