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
google-music-scripts: init at 3.0.0 #53782
Conversation
b29848e
to
211f556
Compare
4a18cfe
to
920913c
Compare
13d6d28
to
6bc8ca7
Compare
@worldofpeace Now that |
Nice, I'll give this a final review to see if there's anything amiss. |
Looks like 6bc8ca7 had a little fixup mishap |
So if a package has no tests we want to explicitly disable |
6bc8ca7
to
0cae3d9
Compare
Yep. Other python maintainers here also agree on this. |
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.
Think it's good 👍
Thanks for your patience @jbaum98
Haven't tested the actual function of the program though.
I'll merge this with the approval of one more maintainer and If someone can vouch for the function.
cc @dotlambda
Note, if I don't get another approval within <24hrs I'll just merge it. |
}; | ||
|
||
# hypothesis-pytest is not needed | ||
patches = [ ./hypothesis_pytest.patch ]; |
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.
I'd prefer doing this in postPatch
using substituteInPlace
.
Did you submit a patch upstream?
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.
I opened a PR, but I actually realized that it's building without the patch.
I can confirm that google-music-scripts is functioning correctly. |
c74286c
to
781f8e1
Compare
781f8e1
to
9237f2b
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.
Remove the patch files.
9237f2b
to
dff4cc6
Compare
dff4cc6
to
bc820fa
Compare
Maybe a little more than 24 hours 🤣 |
No problem, thanks @worldofpeace and @dotlambda for your help! |
Motivation for this change
Add
google-music-scripts
.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)