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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
add geospatial team to maintainers for geospatial packages where I'm a maintainer #288107
Conversation
256965b
to
8e370e7
Compare
8e370e7
to
256965b
Compare
This shouldn't be merged before #288118 is merged, as otherwise I'd appear twice in the expanded lists of maintainers of these packages. Appending "draft: " to the title didn't mark this PR as a draft (I guess that works only on GitLab, not GitHub?), thus temporarily closing this PR here, until #288118 is merged. |
256965b
to
5688d7b
Compare
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 am ok with taking those packages under our maintenance.
@das-g there is a conflict in |
I'll look into it and will rebase. |
5688d7b
to
bdf0b68
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/week-in-geospatial-team/37035/8 |
Description of changes
meta.maintainers
for geospatial packages (and dependencies of geospatial packages) where I (@das-g) am a maintainerThings 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.