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

PEP 676: Implementation updates #2208

Merged
merged 13 commits into from
Jan 9, 2022
Merged

PEP 676: Implementation updates #2208

merged 13 commits into from
Jan 9, 2022

Conversation

AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Jan 3, 2022 •
edited
Loading

This PR has implementation updates for PEP 676.

  • Cache pip in github actions
  • Simplyify parts of generate_rss.py, and remove the dependency on python-dateutil
  • Cache last modified times of each file, speeds up building by 28% on windows (less on unix as subprocess is faster there)
  • Remove pep_sphinx_extensions.config, add the properties to the document object
  • More speed ups with a custom builder, allows removing some patches
  • Remove all uses of javascript
  • Updated PEP 0 lead to inline links (xref Discourage using References? #2130)
  • Fix a false positive for the Docutils inliner in PEP 456

A

@AA-Turner AA-Turner requested review from tiran and a team as code owners January 3, 2022 01:21
@AA-Turner AA-Turner force-pushed the pep-676-implementation-updates branch from c794534 to ef84c09 Compare January 3, 2022 01:39
@AA-Turner AA-Turner mentioned this pull request Jan 3, 2022
Merged
@AA-Turner AA-Turner mentioned this pull request Jan 5, 2022
Merged
@AA-Turner
Copy link
Member Author

@JelleZijlstra thanks for merging #2207, would you mind giving this one a quick review / merge please? (/anyone else I should ask)

A

@JelleZijlstra JelleZijlstra merged commit 3d60b84 into python:main Jan 9, 2022
@AA-Turner AA-Turner deleted the pep-676-implementation-updates branch January 9, 2022 18:35
@CAM-Gerlach CAM-Gerlach mentioned this pull request Jan 29, 2022
Merged
8 tasks
@Do9i

This comment was marked as spam.

Sign in to view
@erlend-aasland erlend-aasland mentioned this pull request May 13, 2022
Closed
@erlend-aasland erlend-aasland mentioned this pull request Jun 27, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hugovk hugovk hugovk approved these changes

@tiran tiran Awaiting requested review from tiran

Assignees
No one assigned
Labels
CLA signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@AA-Turner @Do9i @hugovk @JelleZijlstra @the-knights-who-say-ni