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] oca-check-po: don't check po-pretty-format #274

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

rousseldenis
Copy link
Contributor

@rousseldenis rousseldenis commented Oct 1, 2024

Don't enforce po pretty format that raises if a msgid has the same value as the msgstr. As weblate has those values, it will lead to problems.

Moreover, in some languages, some translations can have the same value as the original language.

@moylop260 @sbidoul

Related to : OCA/server-env#204

Needed for v18.

Don't enforce po pretty format that raises if a msgid has the same
value as the msgstr. As weblate has those values, it will lead
to problems.

Moreover, in some languages, some translations can have the same
value as the original language.
@sbidoul
Copy link
Member

sbidoul commented Oct 1, 2024

The main problem is the error if the translated string is equal to the source string.
To me it is important to not fail in that case, because it happens for legitimate reason, and if we keep the translation empty, weblate will consider that the string is not translated, which is not ok for translators.

@sbidoul sbidoul merged commit 56970b4 into OCA:master Oct 1, 2024
7 checks passed
@sbidoul sbidoul deleted the imp-check-po-dro branch October 1, 2024 13:34
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.

2 participants