Skip to content

Add Dutch (nl) translation agent skill guidelines#62763

Open
subhash-0000 wants to merge 2 commits intoapache:mainfrom
subhash-0000:add-nl-translation-skill
Open

Add Dutch (nl) translation agent skill guidelines#62763
subhash-0000 wants to merge 2 commits intoapache:mainfrom
subhash-0000:add-nl-translation-skill

Conversation

@subhash-0000
Copy link
Contributor

This PR defines the translation agent skill guidelines for the Dutch (nl) locale and adds a corresponding locale-specific README, as part of the initiative to improve AI-assisted localization (#61984).

Changes:

Added

.github/skills/airflow-translations/locales/nl.md
with guidelines on Dutch terminology (e.g., keeping Dag in English, using the informal "je" register).
Added

airflow-core/src/airflow/ui/public/i18n/locales/nl/README.md
to document the rationale for specific term selections for future contributors.
Verified that both files pass core static checks (license headers, trailing whitespace, and end-of-file-fixer).
Reference Issues:

closes: #61999
related: #61984
Was generative AI tooling used to co-author this PR?
Yes (Antigravity/Claude Code)

@boring-cyborg boring-cyborg bot added area:dev-tools area:translations area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch translation:nl labels Mar 3, 2026
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

Technical approval

@subhash-0000
Copy link
Contributor Author

Hi @potiuk @jason810496 — the Firefox UI e2e test is failing in the latest CI run. This PR only adds two Markdown files (nl.md and README.md) with no changes to any Python, TypeScript, or UI code. The same test passed in the previous CI run. Could you please re-run or dismiss this check?

Copy link
Member

@choo121600 choo121600 left a comment

Choose a reason for hiding this comment

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

LGTM
Technical approval

@choo121600
Copy link
Member

Hi @potiuk @jason810496 — the Firefox UI e2e test is failing in the latest CI run. This PR only adds two Markdown files (nl.md and README.md) with no changes to any Python, TypeScript, or UI code. The same test passed in the previous CI run. Could you please re-run or dismiss this check?

Rerun, Thanks ;)

@jason810496 jason810496 removed area:dev-tools area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Mar 4, 2026
Copy link
Member

@jason810496 jason810496 left a comment

Choose a reason for hiding this comment

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

For all Dutch translation owners, I’d appreciate it if you could take a look when you have a moment. Thank you!

cc @BasPH @DjVinnii

@jason810496 jason810496 requested a review from BasPH March 4, 2026 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Define translation agent skill guidelines for Dutch (nl) locale

4 participants