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.

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

Jump to bottom

All PEPs: Move to peps/ folder #3418

Merged
merged 22 commits into from
Sep 9, 2023
Merged

All PEPs: Move to peps/ folder #3418

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift click to select a range
a1cab43
All PEPs: Move to ``peps/`` folder
AA-Turner Sep 1, 2023
18177d9
Update supporting files
AA-Turner Sep 3, 2023
ae247fd
PEPs are now in a subfolder
AA-Turner Sep 3, 2023
a9e08cf
Tests use .rst suffix
AA-Turner Sep 3, 2023
0099c29
Source link is always .rst
AA-Turner Sep 3, 2023
d0e2387
Update paths in Makefile & build.py; fix pre-commit
AA-Turner Sep 3, 2023
3c6a241
Restore executability
AA-Turner Sep 3, 2023
793a746
Notes from review
AA-Turner Sep 3, 2023
16a906a
Further gitignore updates
AA-Turner Sep 3, 2023
12049f8
Keep groups sorted lexicographically (save PEPs)
AA-Turner Sep 3, 2023
c1cd948
Smaller difference
AA-Turner Sep 3, 2023
53f3e2a
Even smaller difference
AA-Turner Sep 3, 2023
5ef3b63
Merge remote-tracking branch 'upstream/main' into peps-in-peps
AA-Turner Sep 3, 2023
14d7bca
Further supporting files updates
AA-Turner Sep 3, 2023
18d1690
Even smaller difference
AA-Turner Sep 3, 2023
89ed8b3
Merge remote-tracking branch 'upstream/main' into peps-in-peps
AA-Turner Sep 5, 2023
8e9727b
Update check-peps
AA-Turner Sep 5, 2023
d43a7c1
Merge remote-tracking branch 'upstream/main' into peps-in-peps
AA-Turner Sep 5, 2023
1e45b3d
whitespace
AA-Turner Sep 5, 2023
1e2fe77
Merge remote-tracking branch 'upstream/main' into peps-in-peps
AA-Turner Sep 6, 2023
74ab33e
Merge remote-tracking branch 'upstream/main' into peps-in-peps
AA-Turner Sep 8, 2023
e6047c1
Merge remote-tracking branch 'upstream/main' into peps-in-peps
AA-Turner Sep 9, 2023
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
Prev Previous commit
Next Next commit
Source link is always .rst
  • Loading branch information
@AA-Turner
AA-Turner committed Sep 3, 2023
commit 0099c2998eb70f07bd89bea713c723c2929dba05
8 changes: 1 addition & 7 deletions pep_sphinx_extensions/pep_processor/html/pep_html_builder.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 1,3 @@
from pathlib import Path

from docutils import nodes
from docutils.frontend import OptionParser
from sphinx.builders.html import StandaloneHTMLBuilder
Expand Down Expand Up @@ -31,10 29,6 @@
except KeyError:
title = ""

# source filename
file_is_rst = Path(self.env.srcdir, docname ".rst").exists()
source_name = f"{docname}.rst" if file_is_rst else f"{docname}.txt"

# local table of contents
toc_tree = self.env.tocs[docname].deepcopy()
if len(toc_tree) and len(toc_tree[0]) > 1:
Expand All @@ -46,8 40,8 @@
else:
toc = "" # PEPs with no sections -- 9, 210

return {"title": title, "sourcename": source_name, "toc": toc, "body": body}
return {"title": title, "toc": toc, "body": body}

Check warning on line 44 in pep_sphinx_extensions/pep_processor/html/pep_html_builder.py

View check run for this annotation

Codecov / codecov/patch

pep_sphinx_extensions/pep_processor/html/pep_html_builder.py#L43-L44 

Added lines #L43 - L44 were not covered by tests

class DirectoryBuilder(FileBuilder):
# sync all overwritten things from DirectoryHTMLBuilder
Expand Down
4 changes: 2 additions & 2 deletions pep_sphinx_extensions/pep_theme/templates/page.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 43,8 @@ <h1>Python Enhancement Proposals</h1>
<h2>Contents</h2>
{{ toc }}
<br>
{%- if not sourcename.startswith(("pep-0000", "topic")) %}
<a id="source" href="https://github.com/python/peps/blob/main/{{sourcename}}">Page Source (GitHub)</a>
{%- if not pagename.startswith(("pep-0000", "topic")) %}
<a id="source" href="https://github.com/python/peps/blob/main/peps/{{pagename}}.rst">Page Source (GitHub)</a>
{%- endif %}
</nav>
</section>
Expand Down
Loading