| 29 May 2023 |
 @piegames:matrix.org | Okay, then I'm struggling to imagine a minor but controversial change in this context | 11:53:53 |
 infinisil | Yeah that wouldn't exist then, any minor change would be accepted by the NAT | 11:54:18 |
| infinisil | In reply to @qyliss:fairydust.space
Would "Smaller changes to the standard may be performed without an RFC" work for you? So I think this wouldn't work for me, because I don't necessarily trust the ~200 committers of nixpkgs to be able to decide what a smaller/uncontroversial change to the standard is | 11:54:51 |
 Alyssa Ross | wouldn't CODEOWNERS etc and our existing social conventions be enough to enforce that? | 11:55:20 |
| infinisil | Yeah probably, I guess it's somewhat implicit | 11:55:58 |
| Alyssa Ross | like I'm not arguing the RFC should say "any committer is free to change how this works at any time" | 11:56:36 |
| infinisil | If the NAT is a codeowner, I'll get notified, and if it's an uncontroversial change and I'm not on vacation I'll accept it | 11:56:58 |
| Alyssa Ross | I'd expect people to wait for your input regardless of whether the RFC says they have to | 11:57:35 |
| Alyssa Ross | and if they don't, well, then we probably have a problem that's wider than just auto-called packages | 11:58:02 |
| infinisil | But then again, I don't think all of our 200 committers are aware of this RFC. It's easy to just randomly pick an RFC to review, decide that it looks trivial, and merge it | 11:58:31 |
| Alyssa Ross | people are generally able to respect soft ownership without needing to know whether an RFC exists | 11:59:40 |
| infinisil | I guess this should maybe go into a separate RFC then, to say that certain teams may take ownership with required approval over agreed-upon parts of Nixpkgs | 11:59:43 |
| Alyssa Ross | or it could just… not, because this has not (to my knowledge) been a big problem with the current system | 12:00:27 |
| @piegames:matrix.org | In reply to @infinisil:matrix.org
I guess this should maybe go into a separate RFC then, to say that certain teams may take ownership with required approval over agreed-upon parts of Nixpkgs Yes, had this thought earlier on. I agree with your distrust towards the general commiters to some extent, but I'm not sure working around that in individual RFCs is the best approach.
This also ties in to the idea that we want a merge bot with a lot more granular access control to the repository …
| 12:01:05 |
| Alyssa Ross | mostly if I ever see changes merged to areas of Nixpkgs that have a clear set of most-knowledgeable people without their approval, it's because somebody has been trying to get their attention for months and hasn't been able to | 12:01:17 |
| infinisil | Hmm.. I've certainly seen certain committers merge changes to core parts of the code that they didn't really know much about, treating it like any other random package update | 12:01:20 |
