potiuk opened a new pull request, #271:
URL: https://github.com/apache/airflow-steward/pull/271
## Summary
- Route every `notifications:` scheme to
`[email protected]` instead of the Airflow PMC's main
`commits@` / `jobs@` lists, so the framework gets its own
dedicated subscribable feed.
- Update surrounding comments: the Airflow PMC's `commits@` and
`jobs@` are now also called out as "do not route here", and the
rationale block reflects the single-list-for-everything design
choice (revisit when this repo moves to its own TLP).
- Rewrite the `copilot_code_review` comment to make explicit that
the directive is the only lever asfyaml exposes for Copilot, so
`enabled: false` is what "disable Copilot completely" reduces to
in this config. The setting itself was already `false`.
## Test plan
- [ ] Confirm `[email protected]` exists / is reachable
before merge (Infra creates / approves new mailing-list
destinations).
- [ ] After merge, watch the next push / PR comment / issue event
to confirm the notification mail lands on `dev-magpie@` and
not on `[email protected]`.
- [ ] Confirm the Copilot review ruleset on the default branch
remains disabled in repo settings.
## Generative-AI disclosure
This change was drafted with Claude Code (Opus 4.7) on the author's
machine; the author reviewed the diff before pushing.
--
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]
