maintainers: add jgarcia#247972
maintainers: add jgarcia#247972chewblacka wants to merge 1 commit intoNixOS:masterfrom chewblacka:master-1
Conversation
Added jgarcia to list of maintainers
|
I don't want to be a pain in the butt here or be offensive, but the commit and PR title do not match the contributing guidelines at all here, so I doubt that we should merge this. Sorry OP, maybe you want to re-read the contributing guidelines and update this PR accordingly? |
Hi many thanks for the reply. If you could give me some guidance on what PR title and commit would be appropriate to add myself to the maintainers list I'd be very grateful. As far as I can work out the CONTRIBUTING.md only mentions package changes, not adding to the maintainer list. Apologies if I've missed something obvious! |
|
If you have a look at the log of the file ( |
Much obliged! I have amended the title. Thanks again. |
|
Yes, please also fix the commit title, that'd be much appreciated! |
You didn't miss anything. I don't think it's mentioned in there. It's in the manual though. |
I'm just going to close this and submit a new request as it's way easier. |
|
That's not necessary, just |
Thanks, next time I'll use the above method. In the mean time I've created a new pull request here: #248028 |
Description of changes
Added jgarcia to list of maintainers
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)