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
pyscroll: init at 2.19.2 #71257
pyscroll: init at 2.19.2 #71257
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.
please pull from github so that we can have some tests, just unittests are good :)
Unfortunately you'll have to match the date to a commit to figure out the version github, as they didn't use a tag/release mechanism there.
Thanks a lot for your feedback. I had some problems with the upstream tests. The |
I would focus on just unit tests, if they make the distinction. |
I have opened an issue regarding the failing tests in the upstream repository bitcraft/pyscroll#34 |
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, merge conflict needs to be resolved though.
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.
Ah, right, the tests...
Seems like the upstream has put work into the tests. Can you give it another try @geistesk? |
Motivation for this change
Add the pyscroll library
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)Notify maintainers
cc @