ytihianine commented on PR #40635:
URL: https://github.com/apache/superset/pull/40635#issuecomment-4986327156
Based on latest AI agent review, here is a description of the issue:
Those translations are not compiling
```
#, no-python-format
msgid "% of parent"
msgstr "% du parent"
#, no-python-format
msgid "% of total"
msgstr "% du total"
```
When running `pybabel compile --use-fuzzy --statistics -d
/app/translations_mo ` in my docker image (using Superset v6.1), I have the
following error:
```
> [superset_custom_img 9/13] RUN pybabel compile --use-fuzzy --statistics
-d /app/translations_mo:
0.715 4934 of 4965 messages (99%) translated in
/app/translations_mo/fr/LC_MESSAGES/messages.po
0.737 error: /app/translations_mo/fr/LC_MESSAGES/messages.po:238:
incompatible format for placeholder 1: 'o' and 'd' are not compatible
0.737 error: /app/translations_mo/fr/LC_MESSAGES/messages.po:242:
incompatible format for placeholder 1: 'o' and 'd' are not compatible
0.737 compiling catalog /app/translations_mo/fr/LC_MESSAGES/messages.po to
/app/translations_mo/fr/LC_MESSAGES/messages.mo
```
The issue is that `%` here should be treated as a character and not as a
placeholder. Should I also update frontend values in this PR or should I make a
new PR to fix it ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
