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.django-simple-captcha: init at 0.5.12 #76515
pythonPackages.django-simple-captcha: init at 0.5.12 #76515
Conversation
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.
each package addition should include the additional entry to python-packages. I would also squash the other "enhance" commit into the first simple-captcha bump. Your commit history should be:
pythonPackages.django-simple-captcha: init at 0.5.6
pythonPackages.django-ranged-response: init at 0.2.0
a8fb31a
to
a026522
Compare
@jonringer Done, commit history adjusted to your requests. |
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.
otherwise LGTM
NIT: the ordering of the packages should probably go in a separate commit, as that's much more likely to cause merge conflicts if these packages needed to be cherry-picked
a026522
to
9fedb1d
Compare
@jonringer all done. |
do you mind resolving conflicts and rebasing? |
co-author: Vladyslav Burzakovskyy <vladislav.burzakovskij@satoshilabs.com>
co-author: Vladyslav Burzakovskyy <vladislav.burzakovskij@satoshilabs.com>
9fedb1d
to
7178664
Compare
@jonringer done |
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
failure unrelated to PR (python38Packages.aiohttp is broken)
[9 built, 9 copied (134.2 MiB), 24.3 MiB DL]
https://github.com/NixOS/nixpkgs/pull/76515
6 package built:
python27Packages.django-ranged-response python27Packages.django-simple-captcha python37Packages.django-ranged-response python37Packages.django-simple-captcha python38Packages.django-ranged-response python38Packages.django-simple-captcha
@GrahamcOfBorg build python27Packages.django-ranged-response python27Packages.django-simple-captcha python37Packages.django-ranged-response python37Packages.django-simple-captcha python38Packages.django-ranged-response python38Packages.django-simple-captcha |
@jonringer FYI: All builds went through |
sorry about that, don't have as much free time as i used to |
@jonringer Don't worry, that's why I pinged you. Glad that was okay. |
Motivation for this change
This PR succeeds #64523 which has somehow stalled and is based on @MrMebelMan's work.
After approval it probably needs to be squashed into 1 commit, but I wanted to keep the history intact so far.
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 @