diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index 9b90d48d143..cbe502c4b99 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -103,3 +103,4 @@ Run all commands from the repo root (where `azldev.toml` lives). If the terminal 4. **Don't edit generated output**: Build artifacts in `base/out/` and `base/build/` are generated (configured by `output-dir`, `log-dir`, `work-dir` in `base/project.toml`) — never edit them directly. Note: `prep-sources -o ` writes to a user-specified directory, separate from these project output dirs. 5. **Mandatory testing**: Any change that affects a component's output (overlays, build config, spec edits, version bumps, new components) MUST be validated by building AND testing the resulting RPMs. A successful build alone is not sufficient — smoke-test in a mock chroot. Pure organizational changes (moving definitions between files, editing descriptions/comments) do not require rebuild. See `AGENTS.md` for the full testing protocol. 6. **New specs use autorelease**: For any spec we author ourselves (local specs), use `%autorelease` and `%autochangelog` rather than a static `Release:` with a hand-written `%changelog`. The default `release.calculation = "auto"` detects ordinary, unconditional `%autorelease` usage, so do not set it explicitly. Use `release = { calculation = "autorelease" }` only when auto-detection cannot identify the spec's autorelease behavior, such as when `%autorelease` is conditional. +7. **Keep overlays resilient to upstream changes**: Express the intended change with minimal dependence on unrelated source details. Avoid unnecessarily matching or reproducing surrounding formatting or content. Use enough context to target the change unambiguously, but no more than needed.