Skip to content

Remove incorrect feature coverage comment from msgfmt script's docstring #112862

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 2 commits into from
Aug 2, 2025

Conversation

m-aciek
Copy link
Contributor

@m-aciek m-aciek commented Dec 8, 2023

msgfmt.py handles plural forms since cb081b8.

The comment was incorrectly applied when adding message context functionality (637a33b) -- original patch was created before the plural forms functionality was merged.

msgfmt handles plural forms since cb081b8.

The comment was incorrectly applied in 637a33b -- original patch was from before the cb081b8 commit.
@m-aciek m-aciek force-pushed the msgfmt-doc-plurals branch from 54e9bfd to 27cce5d Compare December 8, 2023 06:18
@m-aciek m-aciek changed the title Remove feature coverage comment from msgfmt script's docstring Remove incorrect feature coverage comment from msgfmt script's docstring Dec 8, 2023
Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StanFromIreland StanFromIreland requested a review from tomasr8 July 31, 2025 17:37
@tomasr8 tomasr8 merged commit 7475887 into python:main Aug 2, 2025
44 checks passed
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.

4 participants