Skip to content

Update core-dev repo permissions - #41

Merged
rabernat merged 2 commits into
zarr-developers:mainfrom
jhamman:patch-2
Jun 13, 2024
Merged

rabernat merged 2 commits into
zarr-developers:mainfrom
jhamman:patch-2

Conversation

@jhamman

@jhamman jhamman commented May 24, 2024

Copy link
Copy Markdown
Member

Small change to specify what permissions core-dev sub-teams will be granted in the repositories that they manage.

See zarr-developers/zarr-python#1905 and https://github.com/orgs/zarr-developers/discussions/70 for more discussion on the topic.

Comment thread GOVERNANCE.md Resolved
@jhamman

jhamman commented Jun 12, 2024

Copy link
Copy Markdown
Member Author

@zarr-developers/steering-council - what is needed to advance this?

@rabernat rabernat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change seems necessary to allow sub teams to function independently.

@rabernat

Copy link
Copy Markdown
Contributor

what is needed to advance this?

This is a minor change to our governance document. If this change constitutes a "change to the Zarr governance model", then it is a heavy lift

Changes to this governance model or our mission, vision, and values require a dedicated issue on our issue tracker and follow the ZEP process, with "reasonable time" being at least two weeks. However, if there is unanimous agreement from core developers on the change, it can move forward faster.

It seems like massive overkill to make a ZEP here. It also feels like getting unanimous agreement from 38 @zarr-developers/core-devs is overkill.


I do not interpret this PR as any change to the governance model. It's a technical clarification about what permissions core devs should have in GitHub, designed to help us operate more effectively. I see it more as a "Minor documentation change". I'd say that a few approvals and no disagreement is sufficient to merge.

Does anyone disagree with that interpretation?

@d-v-b

d-v-b commented Jun 13, 2024

Copy link
Copy Markdown
Contributor

I agree with your POV @rabernat. As this has been open for nearly a month with no objections, should we try to merge it by the end of the week?

@joshmoore

joshmoore commented Jun 13, 2024 •

Copy link
Copy Markdown
Member

Sorry all. I think this mostly fell through the cracks because the last ZSC meeting got cancelled. Thanks, @jhamman, for opening.

I assume as well that it's a clarification and that everyone has time to object. But of course, it can always be reevaluated if need be.

Merging tomorrow if no last minute objections appear.

@martindurant

Copy link
Copy Markdown
Member

+1

@rabernat
rabernat merged commit c012fe1 into zarr-developers:main Jun 13, 2024
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.

5 participants