-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
treewide: remove srgom #337369
treewide: remove srgom #337369
Conversation
srgom is inactive.
srgom is inactive.
Should they also be removed from the maintainer list? |
I prefer to do this in a future batch. |
It makes sense a bit more to me to do it per maintainer. Other then that, this is good to go. Let's give @SRGOM a few days to respond. If days pass, cc me to merge this if I forget. |
Sometimes some packages are maintained by multiple silent maintainers. It can create some strange dependency trees. Other times some packages are being modified on Staging. Further, some people want to adopt some packages, and I prefer to detach them so that they cam be adopted independently. Because of this and other minor annoying things, I prefer to delete entries in maintainers set a bit later. |
Closing these removal PRs, since they are not grounded within community consensus. See #337478 (comment) @SRGOM be sure that we value your involvement and any time you may have to spare, helping with the packages you're maintaining and others as well. |
That account has been inactive for 4 years. It can be removed with no bad feelings. |
The other PR got a bit hectic, so one could be forgiven for not reading all the way through. So again here in simple terms: I'm not opposed to getting stale maintainers out. I am opposed to doing it the way it's being done in these PRs. More on the new PR you opened, but the gist is: At that rate you could have just re-opened this one ... |
GitHub doesn't allow me to reopen this PR, so I opened a new one. |
Okay...
You are only opposed to the wording of the description of the pull request, correct? Since the PR content was right then. You should have never closed all these PRs, without asking first. |
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.