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
nix-doc: init at 0.3.3 #95161
nix-doc: init at 0.3.3 #95161
Conversation
Apologies about the sneaky edits, there was a pretty nasty bug I found and squished while doing final validation on the package. |
2aec9e1
to
9fbf770
Compare
OK I am done smuggling in changes. It builds and runs now with no issues, and contains two fewer bugs than when I started :) |
@lf- you will be asked to create a separate commit to add yourself as a maintainer as |
You can also ignore that |
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.
Left a few suggestions, other than that LGTM :)
50502e9
to
0220ace
Compare
Thanks for the review @IvarWithoutBones. The requested changes have been made. |
@IvarWithoutBones changed :) |
Motivation for this change
It was requested by @gvolpe :)
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)re: the ??? above, nixpkgs-review did not find anything when I ran it post-commit, and
nix path-info -S
hit the intentionally failing pseudo-package, so I'm not sure how I'm supposed to do that piece.