Skip to content

Fix stale flag name in Airflow translations agent skill - #71332

Merged
jason810496 merged 1 commit into
apache:mainfrom
ColtenOuO:fix-translations-skill-remove-unused-flag
Aug 18, 2026
Merged

Fix stale flag name in Airflow translations agent skill#71332
jason810496 merged 1 commit into
apache:mainfrom
ColtenOuO:fix-translations-skill-remove-unused-flag

Conversation

@ColtenOuO

Copy link
Copy Markdown
Contributor

Sumarry

The airflow-translations agent skill still documents --remove-extra for breeze ui check-translation-completeness.

That option was renamed to --remove-unused in #62983

The same PR merged the separate "extra" and "unused" concepts into a single unused category (keys that are not required — absent from the English locale, or plural suffixes the language does not need), and the summary table now reports Missing / Coverage / TODOs / Unused with no Extra column. This updates both the command and the surrounding wording to match.

Notes

#62983 updated airflow-core/src/airflow/ui/public/i18n/README.md and dev/breeze/doc/10_ui_tasks.rst but missed the skill file.

The translation completeness checker renamed --remove-extra to --remove-unused
and merged the "extra" and "unused" concepts into a single "unused" category,
but the agent skill was left pointing at the old flag. Anyone following the skill
hits a hard "No such option" failure at the cleanup step, and the validation
criteria describe a column that the checker no longer prints.
@potiuk potiuk added the ready for maintainer review Set after triaging when all criteria pass. label Aug 13, 2026

@jason810496 jason810496 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks.

@jason810496
jason810496 merged commit 04e98ca into apache:main Aug 18, 2026
64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for maintainer review Set after triaging when all criteria pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants