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
mgit: Init at 0.1.10 #94173
mgit: Init at 0.1.10 #94173
Conversation
maintainers/maintainer-list.nix
Outdated
@@ -9203,4 +9203,10 @@ | |||
github = "deifactor"; | |||
githubId = 30192992; | |||
}; | |||
koozz = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔡 Users are mostly inserted by alphabetical order. Could you find the next best spot for your username.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, the last few were totally unordered and that made me think it was chronological. I've put myself in the ordered part now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙋 You need to add your package to pkgs/top-level/all-packages.nix
maintainers/maintainer-list.nix
Outdated
name = "Jan van den Berg"; | ||
email = "264371+koozz@users.noreply.github.com"; | ||
github = "koozz"; | ||
githubId = 264371; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the githubId
is off by one 😬
curl -s https://api.github.com/users/koozz | jq .id
264372
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Weird, I found the noreply-email in my GitHub settings and copied that. Fixed it now.
Motivation for this change
Commandline tool written in Rust by myself.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)