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
oneshot: init at 1.0.1 #92792
oneshot: init at 1.0.1 #92792
Conversation
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.
Already looks pretty good! Added some small suggested changes.
Thanks @danieldk for the review. I committed your non-optional changes as requested, so should be ready to merge now. Also I noticed that there is an update to 1.1.0 already available and updated accordingly. I hope that's okay. If you prefer me to submit a second PR for that update, let me know and I will split it. |
That's fine! Could you squash all commits into one and use the following commit message?
|
done |
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.
LGTM
Result of nixpkgs-review pr 92792
1
1 package built:
- oneshot
The build report was served through oneshot
itself before copy-pasting into this post ;).
Motivation for this change
I use the package and want to make it easily available to other nixpkgs users.
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)