-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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.myfitnesspal: init at 1.13.3 #64422
Conversation
@GrahamcOfBorg build python2Packages.myfitnesspal python2Packages.measurement python3Packages.myfitnesspal python3Packages.measurement |
postPatch = '' | ||
sed -i 's/keyring>=.*/keyring/' requirements.txt | ||
sed -i 's/keyrings.alt>=.*/keyrings.alt/' requirements.txt | ||
''; |
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.
Both modules pass all of their test cases, but I also verified manually that the package works; keyring appears to increment major versions very frequently so I just stripped the version bound entirely rather than loosening it to what we currently have.
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.
Kind of a nit pick, but i would just import lib instead of stdenv.
squash commits so there is one per package |
25d3384
to
b0fd855
Compare
Commits now properly squashed per package |
Ping @FRidh, I have squashed the commits as requested. I believe this should be ready for merge, but if not let me know and I'm happy to update further. |
Motivation for this change
This is a python library for downloading data from the MyFitnessPal API, which is useful for users who want to chart/graph/visualize their data with other tools, such as seaborn or matplotlib.
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)