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
chrome-export: init at 2.0.2 #63371
chrome-export: init at 2.0.2 #63371
Conversation
@mmahut Thank you for the review! |
51f9a48
to
b20cdce
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.
This should rather be packaged using mkPythonApplication
rather than mkDerivation
, I guess...
@matthiasbeyer I’m not sure why that would be needed in this case. The package does not have a |
Are there any other concerns with this PR or is it good to be merged? |
This pull request has been mentioned on Nix community. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review-may-2019/3032/27 |
Looking good :) |
Motivation for this change
Adds chrome-export, which converts Chrome’s bookmark and history files to the standard HTML bookmarks file format.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)