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

Use literal syntax in origin property #119029

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

rffontenelle
Copy link
Contributor

@rffontenelle rffontenelle commented May 14, 2024

Without this syntax markup, a translator translating library/importlib.metadata.rst could think that "origin" should be translated when the context shows it is a property name for Distribution class.

The context is:

For editable packages, an origin property may present PEP 610 metadata:

>>> dist.origin.url
'file:///path/to/wheel-0.32.3.editable-py3-none-any.whl'


📚 Documentation preview 📚: https://cpython-previews--119029.org.readthedocs.build/

@hauntsaninja hauntsaninja merged commit 66b73e9 into python:main May 16, 2024
34 checks passed
@miss-islington-app
Copy link

Thanks @rffontenelle for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 16, 2024
(cherry picked from commit 66b73e9)

Co-authored-by: Rafael Fontenelle <rffontenelle@users.noreply.github.com>
@miss-islington-app
Copy link

Sorry, @rffontenelle and @hauntsaninja, I could not cleanly backport this to 3.12 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 66b73e9724fc376715ae264c8282dc1e981e4f17 3.12

@bedevere-app
Copy link

bedevere-app bot commented May 16, 2024

GH-119083 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label May 16, 2024
@hauntsaninja hauntsaninja removed the needs backport to 3.12 bug and security fixes label May 16, 2024
hauntsaninja pushed a commit that referenced this pull request May 16, 2024
Use literal syntax in origin property (GH-119029)
(cherry picked from commit 66b73e9)

Co-authored-by: Rafael Fontenelle <rffontenelle@users.noreply.github.com>
@rffontenelle rffontenelle deleted the patch-2 branch July 11, 2024 07:54
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants