eladkal commented on code in PR #68663: URL: https://github.com/apache/airflow/pull/68663#discussion_r3426845263
########## .agents/skills/prepare-providers-documentation/SKILL.md: ########## @@ -215,8 +215,17 @@ Before spawning a sub-agent, apply the same fast heuristics breeze uses `providers/<id>/src/airflow/providers/<id>/example_dags/**` → **`s`** (skip). - Subject contains `Bump minimum Airflow version` and only `__init__.py` / `provider.yaml` changed → **`v`**. - -Note these classifications and move on — no sub-agent needed. +- CI/infra bumps (`[main] Upgrade important CI environment`, workflow-only + changes) and JavaScript **build/dev tooling** bumps for a bundled UI + (`vite`, `prettier`, `stylelint`, `eslint`, `webpack-*`, `terser-*`, + `@types/*`) → **`s`** (skip / not user-facing), even though they touch files + outside `tests/`. These belong in the hidden block, never in a visible + section. + +Note these classifications and move on — no sub-agent needed. Whatever the +classification, the entry text you ultimately write must follow the +[entry wording rules](#entry-wording-rules--every-entry-must-read-well-for-users) +in Phase 4b — rewrite the subject, don't copy it verbatim. Review Comment: I'd add also specific instruction on `Prepare providers release ...` should always be in the hidden part of the change log as this is internal. -- 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]
