Skip to content

Comments

maintainers: add jgarcia#247972

Closed
chewblacka wants to merge 1 commit intoNixOS:masterfrom
chewblacka:master-1
Closed

maintainers: add jgarcia#247972
chewblacka wants to merge 1 commit intoNixOS:masterfrom
chewblacka:master-1

Conversation

@chewblacka
Copy link
Contributor

@chewblacka chewblacka commented Aug 8, 2023

Description of changes

Added jgarcia to list of maintainers

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.11 Release Notes (or backporting 23.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Added jgarcia to list of maintainers
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Aug 8, 2023
@matthiasbeyer
Copy link
Contributor

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?

@chewblacka
Copy link
Contributor Author

chewblacka commented Aug 8, 2023

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!

@matthiasbeyer
Copy link
Contributor

If you have a look at the log of the file (git log --oneline -- maintainers/maintainer-list.nix), you'll see that commonly the commit message is maintainers: add <username>. That'd be a perfect PR title in this case as well.

@chewblacka chewblacka changed the title Update maintainer-list.nix maintainers: add jgarcia Aug 8, 2023
@chewblacka
Copy link
Contributor Author

If you have a look at the log of the file (git log --oneline -- maintainers/maintainer-list.nix), you'll see that commonly the commit message is maintainers: add <username>. That'd be a perfect PR title in this case as well.

Much obliged! I have amended the title. Thanks again.

@matthiasbeyer
Copy link
Contributor

Yes, please also fix the commit title, that'd be much appreciated!

@ius
Copy link
Contributor

ius commented Aug 8, 2023

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!

You didn't miss anything. I don't think it's mentioned in there. It's in the manual though.

@chewblacka
Copy link
Contributor Author

Yes, please also fix the commit title, that'd be much appreciated!

I'm just going to close this and submit a new request as it's way easier.

@chewblacka chewblacka closed this Aug 8, 2023
@matthiasbeyer
Copy link
Contributor

That's not necessary, just git commit --amend the commit you pushed here and git push this with a little bit of --force.

@chewblacka
Copy link
Contributor Author

That's not necessary, just git commit --amend the commit you pushed here and git push this with a little bit of --force.

Thanks, next time I'll use the above method. In the mean time I've created a new pull request here: #248028

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants