Add initial Portuguese (pt) translation agent skill (#62001)#62083
Add initial Portuguese (pt) translation agent skill (#62001)#62083jason810496 merged 12 commits intoapache:mainfrom
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
|
Hi @jason810496, Opened this as a draft PR with the initial pt.md content based on the existing Portuguese JSON files. Happy to adjust any vocabulary, add/remove examples, or refine the rules based on feedback from Portuguese maintainers. Thanks again for the assignment and guidance! |
There was a problem hiding this comment.
Thanks for helping out @Jayantparashar10!
We still need to resolve the static check error based on GitHub Action CI before merging the PR.
I will recommend to install prek (uv tool install prek==0.3.3) to resolve all the errors automatically.
Thanks!
|
Hi @jason810496, Static checks are now passing locally and should be same on CI (added blank lines around lists, license header confirmed). Ready for final merge when convenient. Thanks! |
jason810496
left a comment
There was a problem hiding this comment.
Thanks for the update!
There's only a final nit, then the PR is good to go.
|
Hi @jason810496, Thanks for the quick review I’ve now updated
Ready for merge whenever you have a moment. Thanks again! |
There was a problem hiding this comment.
Pull request overview
Adds an initial Portuguese (pt) “agent skill” guideline document to help AI-assisted UI translations remain consistent with existing Airflow pt locale JSON strings.
Changes:
- Add
.github/skills/airflow-translations/locales/pt.mdwith Portuguese translation guidance and examples aligned to existingpt/*.jsonfiles.
|
Hi @jason810496, I believe the PR is now ready to merge whenever you have a moment. Thanks! |
|
I will merge after the CI green, thanks for the help @Jayantparashar10! |
|
Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions. |
…pache#62083) * Add initial Portuguese (pt) translation agent skill (apache#62001) * Update pt.md and remove SKILL.md * Fix markdownlint MD032: add blank lines around lists + confirm license header (apache#62001) * Remove long ASF license header as per request * Fix insert-license CI error --------- Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
…pache#62083) * Add initial Portuguese (pt) translation agent skill (apache#62001) * Update pt.md and remove SKILL.md * Fix markdownlint MD032: add blank lines around lists + confirm license header (apache#62001) * Remove long ASF license header as per request * Fix insert-license CI error --------- Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
…pache#62083) * Add initial Portuguese (pt) translation agent skill (apache#62001) * Update pt.md and remove SKILL.md * Fix markdownlint MD032: add blank lines around lists + confirm license header (apache#62001) * Remove long ASF license header as per request * Fix insert-license CI error --------- Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
Description
This PR creates the Portuguese (pt) translation agent skill file as part of the broader initiative to help AI agents produce consistent translations for Airflow's UI.
Changes
.github/skills/airflow-translations/locales/pt.mdwith comprehensive Portuguese translation guidelines.github/skills/airflow-translations/SKILL.mdfor global terminology rules (to be completed in Define global-level translation agent skills #61985)airflow-core/src/airflow/ui/public/i18n/locales/pt/*.jsonfilesContent Overview
The
pt.mdskill file includes:Testing Strategy
After merge, any AI agent supporting "skills" context can:
Example test case:
"retry_delay": "Retry delay in seconds""retry_delay": "Atraso de retentativa em segundos"Alignment
Closes: #62001
Was generative AI tooling used to co-author this PR?