andreahlert commented on code in PR #62983:
URL: https://github.com/apache/airflow/pull/62983#discussion_r2894892413
##########
dev/breeze/src/airflow_breeze/commands/ui_commands.py:
##########
@@ -88,7 +88,7 @@ def char_key(c: str) -> tuple:
"ko": ["_other"],
"nl": MOST_COMMON_PLURAL_SUFFIXES,
"pl": ["_one", "_few", "_many", "_other"],
- "pt": ["_zero", "_one", "_many", "_other"],
+ "pt": MOST_COMMON_PLURAL_SUFFIXES,
Review Comment:
Thanks for flagging this. It wasn’t intentional.
I’ll revert pt here to only:
"pt": ["_zero", "_one", "_many", "_other"]
I started working on the translation checker while evaluating the pt-BR
translations, and that’s how the duplicate/conflicting pt entry slipped in. The
change to use MOST_COMMON_PLURAL_SUFFIXES (or similar) for Portuguese is
something I’m handling in PR #62965, so I’ll keep this PR focused and leave the
plural-suffix logic for Portuguese to that PR.
--
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]
