Skip to content

doc/nixpkgs-committers: Link instead - #276

Open
roberth wants to merge 1 commit into
NixOS:mainfrom
roberth:committers-policy-link
Open

doc/nixpkgs-committers: Link instead#276
roberth wants to merge 1 commit into
NixOS:mainfrom
roberth:committers-policy-link

Conversation

@roberth

@roberth roberth commented Aug 5, 2026

Copy link
Copy Markdown
Member

Document the policy the Nixpkgs Core team has adopted, by link.

This is technically a change in policy, hence the request for @NixOS/steering reviews

@roberth
roberth requested a review from a team August 5, 2026 16:32
@roberth
roberth requested a review from a team as a code owner August 5, 2026 16:32
@K900

K900 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

This file is generally managed by the nixpkgs core team, not SC.

@roberth

roberth commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

Not sure what you mean @K900. The Nixpkgs Core team has only adjusted the "team" part of this page.
This PR reinforces the idea that the core team manages the commit bit policy, but by referring to "this file" you are either supporting the PR, or presenting factually incorrect information. I suppose you meant to support the PR, but you gave a comment, not an approval. I'm confused.

@K900

K900 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

What I'm saying is that SC should not be making any changes here without explicit core team approval, and generally should not be initiating meaningful changes ever.

@mweinelt

mweinelt commented Aug 5, 2026

Copy link
Copy Markdown
Member

You requested the SC for review even though it does not own this file, that's what seemed weird to me. The SC approvals are meaningless.

@roberth

roberth commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

I'm doing this entirely to help the core team. Let me fix the request for review

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