Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify and extend org owner authority #61

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

infinisil
Copy link
Member

@infinisil infinisil commented Jan 22, 2025

I've been annoying the other org members with my complaints when we didn't follow the authority to the dot 😅

This was because:

  • The "such as" wording left it open to ambiguity as to whether it's only the listed items or also others (while proper English would imply the latter, I meant the former when I wrote this originally)
  • The former interpretation is unnecessarily restrictive, requiring further requests to increase authority whenever something new was being done.

This PR makes it very explicit that it's an exclusive list for individually-implementable changes (certain pre-approved changes and minimal-impact changes), while leaving all other non-controversial and non-impactful changes up to the org owners, given two approvals.

This should hopefully require less changes to this document in the future.

@infinisil infinisil requested a review from a team as a code owner January 22, 2025 18:56
- The "such as" wording left it open to ambiguity as to whether it's
  only the listed items or also others (while proper English would imply the
  latter, I meant the former when I wrote this originally)
- The former interpretation is unnecessarily restrictive, requiring
  further requests to increase authority whenever something new was
  being done.

This change makes it very explicit that it's an exclusive list for
individually-implementable changes (certain pre-approved changes and
minimal-impact changes), while leaving _all_ other non-controversial and
non-impactful changes up to the org owners, given two approvals.
@infinisil infinisil force-pushed the org-owner-authority-improved branch from 3397118 to addc735 Compare January 22, 2025 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant