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
pythonPackages.haas-nabucasa: init at 0.29 #75876
Conversation
a05d36f
to
47a7785
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.
diff LGTM
commits LGTM
has tests
[4 built, 2 copied (0.0 MiB), 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/75876
2 package were built:
python37Packages.snitun python38Packages.snitun
just some small suggestions
@GrahamcOfBorg build python37Packages.snitun python38Packages.snitun |
@jonringer I added another package to this pr since it depends on the original one, or should I open a new pull request for the next one? |
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.
your commit history should be:
pythonPackages.snitun: init at 0.20
pythonPackages.haas-nabucasa: init at 0.29
I would recommend using git rebase -i
it's fine, just need to update your PR to describe what you're doing |
8b56dbc
to
9cceab6
Compare
you have snitun changes in the pythonPackages.haas-nabucasa commit do the following to fix:
|
9cceab6
to
7b4d5a0
Compare
pythonPackages.haas-nabucasa: init at 0.29 (cherry picked from commit 45e69f1)
@Scriptkiddi can this be moved inside the As elaborated in NabuCasa/hass-nabucasa#119 (comment), upstream "doesn't care about other ways to run Home Assistant" (except docker and virtualenv install), so we should probably not have it in the curated python package set, but move it to https://github.com/NixOS/nixpkgs/blob/master/pkgs/servers/home-assistant/default.nix, (with some overrides, so this module doesn't block |
I will do that, I will also see if we can not remove such a module upstream from the hass cloud module so its not required, lets see how good my chances are seeing that those guys from nabucasa started hass .... |
Motivation for this change
Things done
nix-shell -p nix-review --run "nix-review wip"
sandbox
innix.conf
on non-NixOS linux)./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @