Delete CODEOWNERS to eliminate notification overload #1413
Merged
+0
−23
Conversation
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
Only auth-related files now have explicit ownership to reduce unnecessary review requests on every PR while maintaining security oversight where needed. Reported-by: User
The CODEOWNERS file was causing excessive notifications by automatically requesting reviews from entire teams on every PR. Since we have no branch protection rules enforcing CODEOWNERS approval, this file only served to spam team members without providing actual merge protection. Going forward: - Reviews will be triaged through manual review requests and notifications - Team members with write access can still merge as before - This eliminates automatic review requests while maintaining the same security posture (no enforcement was in place anyway) This significantly reduces notification fatigue for the python-sdk team (6 members) and python-sdk-auth team (2 members) who were being auto-assigned to every relevant PR. Reported-by: User
felixweinberger
requested review from
a team,
dsp-ant,
jerome3o-anthropic and
jspahrsummers
and removed request for
jspahrsummers,
jerome3o-anthropic and
dsp-ant
|
@pcarleton @ochafik FYI - based on offline conversation it sounds like we weren't gaining much value from these notifs in the first place, so going to try a different approach |
Kludex
approved these changes
Sep 29, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Removes CODEOWNERS file that was causing unnecessary reviewer notification spam without providing actual merge protection.
Problem
Solution
Delete CODEOWNERS entirely. Reviews will be handled through manual review requests based on PR content.