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

Fix compose output of extracted comments on plural messages #133

Merged

Conversation

mthiems
Copy link
Contributor

@mthiems mthiems commented Mar 26, 2024

Fixes #132

  • Adds tests
    • Without the fix, two of the new tests fail
  • One-line fix
    • With the fix, all tests pass: 38 doctests, 349 tests, 0 failures

I've also verified that this addresses the issues I am having with gettext.extract and gettext.merge when there are extracted comments on plural messages in .pot/.po files (as mentioned in #132).

for extracted comments and plural messages

Two test failures as of this commit:
- compose/2 plural message with extracted comments
- compose/2 plural message with both comments and extracted comments
Correct `compose` output of extracted comments on plural messages
@whatyouhide
Copy link
Contributor

Good catch, thank you @mthiems! 💟

@whatyouhide whatyouhide changed the title Correct compose output of extracted comments on plural messages Fix compose output of extracted comments on plural messages Mar 26, 2024
@whatyouhide whatyouhide enabled auto-merge (squash) March 26, 2024 12:50
@coveralls
Copy link

Pull Request Test Coverage Report for Build f718fbda496c281f0e0c559f113575d049e8cebb-PR-133

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.296%

Totals Coverage Status
Change from base Build dbcc3e2a31f4c4b7f861edd39813ac32f851b7ed: 0.0%
Covered Lines: 423
Relevant Lines: 426

💛 - Coveralls

@whatyouhide whatyouhide merged commit 4d168bd into elixir-gettext:main Mar 26, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extracted comments on plural messages output as regular comments by compose
3 participants