chore(asf.yaml): refresh collaborators list and keep approvals through pushes - #1116
Merged
Conversation
choo121600
approved these changes
Aug 26, 2026
3 tasks
Member
Author
|
And this one to fix the ruff check #1117 |
…h pushes `collaborators:` is trimmed to the non-ASF-members named in MISSION.md: `johnslavik`, plus `shahar1`, `vincbeck` and `bugraoz93` — the three Airflow committers from that section, now active entries instead of commented-out records of intent. `choo121600` and the eight non-Airflow-committer ASF members from the PMC roster are removed; that block's comment goes with them. `dismiss_stale_reviews` flips to `false`, so an approval survives later pushes to the branch — a rebase or a review-comment fixup no longer forces a re-approval round-trip. Both surrounding comment blocks are rewritten to describe the values that are actually in the file.
potiuk
force-pushed
the
refresh-asf-yaml
branch
from
August 26, 2026 11:36
50e7ec1 to
cd3d3fc
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
collaborators:is trimmed to the non-ASF-members named in MISSION.md —johnslavik, plusshahar1,vincbeckandbugraoz93, the three Airflowcommitters from that section, now active entries rather than commented-out
records of intent.
choo121600and the eight non-Airflow-committer ASFmembers from the PMC roster are removed. The list goes from 9 entries to 4,
well under ASF Infra's cap of 10.
dismiss_stale_reviewsflips tofalse, so an approval survives later pushesto the branch: a rebase or a review-comment fixup no longer forces a
re-approval round-trip.
actually in the file — they previously contradicted the config.
Type of change
prek, workflows, validators)Test plan
prek run --files .asf.yamlpasses.asf.yamlparses as valid YAMLfile on merge to
main.RFC-AI-0004 compliance
Not applicable — repository configuration only, no skill, tool, or agent
behaviour changes.
Linked issues
Notes for reviewers
The dropped PMC-roster members lose the explicit triage grant this file gave
them; those with
apacheGitHub org access keep reach through that route.Worth a second pair of eyes on whether that trade is intended before merge.