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

gh-126529: Update devguide links to relative filenames in InternalDocs/parser.md and InternalDocs/compiler.md #126530

Merged
merged 2 commits into from
Nov 7, 2024

Conversation

valerii-chirkov
Copy link
Contributor

@valerii-chirkov valerii-chirkov commented Nov 7, 2024

Mentioned it for parsed.md in this Issue #126509

Closed in this PR #126510

Found the same issue in compiler.md, and searched through InternalDocs for any other appearance of devguide links, found nothing except two in compiler.md ("Guide to the parser" and "Changing CPython’s grammar").
Also mentioned that in this PR #126510 I added a relative link as ./, but in InternalDocs only filename as a path is used, so changed it from ./changing_grammar.md to changing_grammar.md

@vstinner vstinner merged commit 19c2481 into python:main Nov 7, 2024
27 checks passed
@vstinner vstinner added the needs backport to 3.13 bugs and security fixes label Nov 7, 2024
@miss-islington-app
Copy link

Thanks @valerii-chirkov for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Sorry, @valerii-chirkov and @vstinner, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 19c248185343dfad046bbe4046b2b900e7405666 3.13

@vstinner vstinner removed the needs backport to 3.13 bugs and security fixes label Nov 7, 2024
@vstinner
Copy link
Member

vstinner commented Nov 7, 2024

Merged, thanks for your contribution!

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.

3 participants