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
nixos/pinnwand: init; steck: init at 0.5.0; nixos/tests/pinnwand: init #89444
Conversation
4633ec2
to
545ce6e
Compare
Hm, since moving to pyproject format in 545ce6e I've lost Edit: Apparently I lost everything 😕. The pyproject file is here https://github.com/supakeen/pinnwand/blob/master/pyproject.toml.
|
545ce6e
to
8f21f87
Compare
8f21f87
to
e177354
Compare
@FRidh Can I kindly ask you to look into this? You seem to be familiar with the pyproject format. #89444 (comment) |
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.
python packaging LGTM
not sure about service/test
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 good. A few minor comments but overall very nice module.
@jonringer Except the package is empty, see #89444 (comment). |
I think its not working because |
That is spot on, that made it work. Thank you. |
d4b3266
to
e255e97
Compare
Please report the issue upstream because it is a bug causing other front-ends aside from poetry (such as pip) to fail. edit: see now that you've done that, thanks! supakeen/pinnwand#93 |
Upstream issue at supakeen/pinnwand#93. Talked to the maintainer directly about this, therefore the issue is rather brief. |
89cd67a
to
beb2ed9
Compare
Packaging issue resolved with 1.2.1. |
@GrahamcOfBorg test pinnwand |
Build the package from Git using pyproject.toml and enable tests.
dfb8943
to
45d5dc2
Compare
Motivation for this change
Adds
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)