Skip to content

GH-121970: Fix gettext for audit events #122651

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

Merged
merged 1 commit into from
Aug 3, 2024

Conversation

AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Aug 3, 2024

cc @rffontenelle

This improves upon the previous situation slightly, as now we don't output strings for translation that will never be used (e.g. sys.addaudithook, where the audit-event directive has content.

Fixes #122629.

A


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

@AA-Turner AA-Turner added docs Documentation in the Doc dir skip news needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Aug 3, 2024
@AA-Turner AA-Turner merged commit 95f5c89 into python:main Aug 3, 2024
29 checks passed
@miss-islington-app
Copy link

Thanks @AA-Turner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 3, 2024
(cherry picked from commit 95f5c89)

Co-authored-by: Adam Turner <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 3, 2024
(cherry picked from commit 95f5c89)

Co-authored-by: Adam Turner <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Aug 3, 2024

GH-122653 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 Aug 3, 2024
@AA-Turner AA-Turner deleted the docs/audit-event-gettext branch August 3, 2024 16:41
@bedevere-app
Copy link

bedevere-app bot commented Aug 3, 2024

GH-122654 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Aug 3, 2024
AA-Turner added a commit that referenced this pull request Aug 3, 2024
GH-121970: Fix ``gettext`` for audit events (GH-122651)
(cherry picked from commit 95f5c89)

Co-authored-by: Adam Turner <[email protected]>
brandtbucher pushed a commit to brandtbucher/cpython that referenced this pull request Aug 7, 2024
blhsing pushed a commit to blhsing/cpython that referenced this pull request Aug 22, 2024
Yhg1s pushed a commit that referenced this pull request Sep 2, 2024
GH-121970: Fix ``gettext`` for audit events (GH-122651)
(cherry picked from commit 95f5c89)

Co-authored-by: Adam Turner <[email protected]>
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 news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translation strings for auditing events no longer have reST markup
1 participant