potiuk opened a new pull request, #67149: URL: https://github.com/apache/airflow/pull/67149
## Summary Refreshes the committed `apache-steward` bootstrap copy at `.github/skills/setup-steward/` to match a newer framework snapshot (~5 weeks of upstream changes), and evolves the local override for `pr-management-triage` comment templates in step with framework PR [apache/airflow-steward#226](https://github.com/apache/airflow-steward/pull/226) (now merged). Three commits, each independently reviewable: 1. **`8ea475a3e8` — Refresh apache-steward bootstrap to snapshot b20a3f2** The committed bootstrap copy of `setup-steward` was at the framework version that landed with the original adoption. This commit re-syncs it to snapshot `b20a3f2`, picking up 11 framework commits — most relevant for this repo: the new `issue` opt-in family (apache-steward#209), `pr-management-triage` prefetch improvements (apache-steward#211), and `pr-management-stats` predicate fixes (apache-steward#219). No opt-in family changes for this repo (still just `pr-management`). 2. **`453fe90fd4` — Drop @-ping of reviewer in author-primary triage templates** Local override: when a triage comment's only addressee is the PR author (`request-author-confirmation`, `reviewer-ping` author-primary, `review-nudge` author-primary), reference the reviewer with backtick-quoted login (no `@`) so the bot's message doesn't notify the reviewer when the next action is on the author. Also adds explicit "resolve threads + ping the reviewer when ready" hand-back instructions. 3. **`b9eb9c46c7` — Drop redundant author-primary template overrides — now framework default** The local override pattern from commit 2 was upstreamed in apache-steward#226 (merged). Removes the four sections of the override that are now identical to the framework's default, leaving only the Airflow-specific content (URLs, AI footer wording, project-specific template bodies). Override file shrinks 279 → 220 lines. ## What landed in the framework that this PR depends on apache-steward main now includes: - New `<reviewer_logins>` placeholder (backtick-quoted login, no `@`-ping) for author-primary templates - Updated `request-author-confirmation`, `reviewer-ping` (author-primary), and `review-nudge` (author-primary) templates using the new placeholder + resolve-and-ping pattern - Rows 19/20/F4 of `pr-management-triage`'s decision table now ignore non-collaborator unresolved threads when deciding `passing → mark-ready` (was: any unresolved thread blocked it) ## Test plan - [x] `/setup-steward verify` clean on the new snapshot. - [x] `/pr-management-triage` ran ~50 actions across pages 2-9 of the open-PR queue using the new templates (no stray @-mentions to reviewers in author-primary contexts). - [x] `/pr-management-stats` HTML dashboard renders the contributor PR funnel correctly against the new snapshot. - [x] No CI files touched; no `airflow-core` / `task-sdk` / `providers/` changes — purely tooling. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 4.7) Generated-by: Claude Code (Opus 4.7) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
