Skip to content

Add initial Portuguese (pt) translation agent skill (#62001)#62083

Merged
jason810496 merged 12 commits intoapache:mainfrom
Jayantparashar10:add-pt-translation-skill
Mar 26, 2026
Merged

Add initial Portuguese (pt) translation agent skill (#62001)#62083
jason810496 merged 12 commits intoapache:mainfrom
Jayantparashar10:add-pt-translation-skill

Conversation

@Jayantparashar10
Copy link
Copy Markdown
Contributor

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

  • Creates .github/skills/airflow-translations/locales/pt.md with comprehensive Portuguese translation guidelines
  • Adds minimal placeholder .github/skills/airflow-translations/SKILL.md for global terminology rules (to be completed in Define global-level translation agent skills #61985)
  • Provides real-world examples extracted from existing airflow-core/src/airflow/ui/public/i18n/locales/pt/*.json files

Content Overview

The pt.md skill file includes:

  1. Core Airflow Terminology - Terms that must remain in English (Dag, Asset, Backfill, XCom, etc.)
  2. Portuguese-Specific Guidelines - Gender agreement, plural forms, capitalization, spelling conventions
  3. Translation Examples - Real patterns from existing JSON files showing correct translations
  4. Agent Instructions - Clear DO/DON'T rules for AI translation agents

Testing Strategy

After merge, any AI agent supporting "skills" context can:

  1. Load this file as reference material
  2. Receive new English UI strings for translation
  3. Output Portuguese translations matching existing style/conventions

Example test case:

  • Input: "retry_delay": "Retry delay in seconds"
  • Expected: "retry_delay": "Atraso de retentativa em segundos"
  • Validation: Matches Portuguese style from existing files

Alignment

Closes: #62001

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

@boring-cyborg
Copy link
Copy Markdown

boring-cyborg bot commented Feb 17, 2026

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)
Here are some useful points:

  • Pay attention to the quality of your code (ruff, mypy and type annotations). Our prek-hooks will help you with that.
  • In case of a new feature add useful documentation (in docstrings or in docs/ directory). Adding a new operator? Check this short guide Consider adding an example DAG that shows how users should use it.
  • Consider using Breeze environment for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
  • Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
  • Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
  • Be sure to read the Airflow Coding style.
  • Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
    Apache Airflow is a community-driven project and together we are making it better 🚀.
    In case of doubts contact the developers at:
    Mailing List: dev@airflow.apache.org
    Slack: https://s.apache.org/airflow-slack

@Jayantparashar10
Copy link
Copy Markdown
Contributor Author

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!

Copy link
Copy Markdown
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 Portuguese translation owners, I’d appreciate it if you could take a look when you have a moment. Thank you!

cc @aoelvp94 @victoru2

Copy link
Copy Markdown
Contributor

@aoelvp94 aoelvp94 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
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.

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!

@Jayantparashar10
Copy link
Copy Markdown
Contributor Author

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!

Copy link
Copy Markdown
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.

Thanks for the update!
There's only a final nit, then the PR is good to go.

@Jayantparashar10
Copy link
Copy Markdown
Contributor Author

Hi @jason810496,

Thanks for the quick review

I’ve now updated pt.md:

  • Removed the long ASF license block and the entire Table of Contents
  • Replaced with the short SPDX header (exactly like in zh-CN.md)

Ready for merge whenever you have a moment.

Thanks again!

Copilot AI review requested due to automatic review settings February 27, 2026 05:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.md with Portuguese translation guidance and examples aligned to existing pt/*.json files.

@jason810496 jason810496 requested review from kaxil and potiuk as code owners March 4, 2026 08:51
@Jayantparashar10
Copy link
Copy Markdown
Contributor Author

Hi @jason810496,

I believe the PR is now ready to merge whenever you have a moment.

Thanks!

@jason810496
Copy link
Copy Markdown
Member

I will merge after the CI green, thanks for the help @Jayantparashar10!

@jason810496 jason810496 merged commit ac9b401 into apache:main Mar 26, 2026
64 checks passed
@boring-cyborg
Copy link
Copy Markdown

boring-cyborg bot commented Mar 26, 2026

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Mar 30, 2026
…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>
Suraj-kumar00 pushed a commit to Suraj-kumar00/airflow that referenced this pull request Apr 7, 2026
…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>
abhijeets25012-tech pushed a commit to abhijeets25012-tech/airflow that referenced this pull request Apr 9, 2026
…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>
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 Portuguese (pt) locale

4 participants