potiuk commented on code in PR #64012: URL: https://github.com/apache/airflow/pull/64012#discussion_r3172251658
########## .agents/skills/stage-changes/SKILL.md: ########## @@ -0,0 +1,72 @@ +<!-- SPDX-License-Identifier: Apache-2.0 + https://www.apache.org/licenses/LICENSE-2.0 --> + +<!-- markdownlint-disable MD022 --> +--- +name: stage-changes +description: Stage changed files for commit using git add. This is a host-only operation — it must never run inside a Breeze container. +--- +<!-- markdownlint-enable MD022 --> + +Stage Changes +============= + +Stage specific files or directories before running static checks or committing. + +Source of truth for contribution workflow order is `contributing-docs`; this skill +enforces the host/container guardrails when applying that workflow. + +Host-Only Guardrail +------------------- + +This skill operates on the **host** working tree. Running `git` commands inside a +Breeze container will silently operate on container-local paths that differ from your +actual working tree. + +**Before running:** Confirm you are on the host by checking: + +- `/.dockerenv` does **not** exist Review Comment: I am not sure those guardrails here are really needed - and I think they are just somewhat wrong BREEZE="true" should be all that is needed. I am not sure where `BREEZE_HOME` is from - it seems only added in tests here and is not actually set in any other place, so I am not sure how it would work ? -- 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]
