Skip to content

maintainers: add msgilligan#421885

Merged
wolfgangwalther merged 2 commits intoNixOS:masterfrom
msgilligan:msgilligan/maintainers-add-msgilligan
Jul 3, 2025
Merged

maintainers: add msgilligan#421885
wolfgangwalther merged 2 commits intoNixOS:masterfrom
msgilligan:msgilligan/maintainers-add-msgilligan

Conversation

@msgilligan
Copy link
Contributor

@emmanuelrosa will be adding me as a maintainer of sparrow, so please add me to the maintainers list.

@nixpkgs-ci nixpkgs-ci 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. 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` labels Jul 2, 2025
@wolfgangwalther
Copy link
Contributor

We generally want to add the maintainers-list entry in the same PR where you're added to that package as a maintainer. That's because otherwise we risk merging this, never coming around to do the other PR - and then having an unused maintainer entry.

Maybe @emmanuelrosa can cherry-pick this into their PR if it already exists? If it does not exist, yet - @msgilligan you can just add yourself as a maintainer to sparrow in a second commit in here.

@msgilligan
Copy link
Contributor Author

If it does not exist, yet - @msgilligan you can just add yourself as a maintainer to sparrow in a second commit in here.

Done.

Copy link
Contributor

@wolfgangwalther wolfgangwalther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diff LGTM, waiting for existing sparrow maintainers to give feedback on the addition.

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jul 2, 2025
Copy link
Contributor

@emmanuelrosa emmanuelrosa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Jul 2, 2025
@wolfgangwalther
Copy link
Contributor

Welcome to Nixpkgs!

@wolfgangwalther wolfgangwalther merged commit a76485d into NixOS:master Jul 3, 2025
27 of 29 checks passed
@msgilligan
Copy link
Contributor Author

Welcome to Nixpkgs!

Thank you!

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

Labels

8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 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. 12.approvals: 2 This PR was reviewed and approved by two persons.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants