-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
python3Packages.avro-python3: init at 1.8.2 #77379
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
Conversation
src = fetchPypi { | ||
inherit pname version; | ||
sha256 = "f82cf0d66189600b1e6b442f650ad5aca6c189576723dcbf6f9ce096eab81bd6"; | ||
}; |
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.
looks like the source distribution doesn't contain tests, it would be prefereable to check out from git hub. E.g fetchFromGitHub
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.
If we try to fetch from github, then lot of other languages(like java, c, c++, ...) code is coming, which is not relevant.
also, commit message should be:
|
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.
the commit history should be:
maintainers: add ktrsoft
python3Packages.avro-python3: init at 1.8.2
Can you please let me know, is there a way to change now? |
@ktrsoft See "Amending the message of older or multiple commit messages" on https://help.github.com/en/github/committing-changes-to-your-project/changing-a-commit-message |
@jonringer Fixed the commit messages as requested |
@shlevy Thanks |
whats the difference between avro and avro-python3? they seem to be at the same version number |
avro is for python2 |
python3Packages.avro-python3: init at 1.8.2 (cherry picked from commit 244c89d)
Motivation for this change
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)Notify maintainers
cc @