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
python3Packages.swspotify: init at 1.2.1, swaglyrics: init at 1.2.2 #96051
Conversation
@GrahamcOfBorg build swaglyrics |
Converted to draft since running the executable fails due to the following error. Package author has been contacted and I'll await new release on PyPI.
|
a2af2c3
to
962be4b
Compare
@GrahamcOfBorg build swaglyrics |
962be4b
to
26bd8d7
Compare
93097b4
to
5772929
Compare
@GrahamcOfBorg build swaglyrics |
Ok, rebased on latest master. Enabling tests still causes the build to fail, though. |
if they still have unit tests that run within the sandbox, they should be enabled as much as possible. If all the tests aren't able to run, then there should be a comment stating that they aren't able to be ran within a sandbox. |
I think I realize what's happening now by testing with |
116dec5
to
a98dbc0
Compare
@jonringer I've updated the derivation to disable tests that require network access, need to do the same with the |
15b7693
to
0eae2c6
Compare
+++ b/tests/test_tab.py | ||
@@ -4,6 +4,7 @@ Contains unit tests for tab.py | ||
import flask_testing | ||
from SwSpotify import SpotifyNotRunning | ||
from mock import patch | ||
+import unittest |
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 patch should be unnecessary with my other comment
7708e28
to
ac47400
Compare
95f9f0d
to
4e9cdd7
Compare
4e9cdd7
to
c159b1f
Compare
d1f8cf7
to
8636842
Compare
cb16bea
to
56ae3c3
Compare
Just tested on NixOS, tests seem to pass and package runs as expected. |
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 96051 1
3 packages built:
- python37Packages.swspotify
- python38Packages.swspotify
- swaglyrics
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)