-
-
Notifications
You must be signed in to change notification settings - Fork 370
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Adding test for pypi upload token #170
Conversation
Hey 👋 You can test locally with Also, thanks so much for contributing! :D |
Thanks so much @bee-san. I ran tests on local and they seem to be passing. When I look at the details of the tests that are failing, all seem to have to do with Git diff after re-formatting with black (shows that there are other tests that have not been formatted)? |
Ah I think I found the issue. There was some spacing that black did not like. I've pushed another change. Hopefully this works. 🤞 |
If you run
|
Thanks @bee-san! I was using the wrong version of black. Formatted it using the right version and in the poetry env. Hopefully it'll work this time. Appreciate the patient responses! |
Wooo! Thanks for contributing :D |
Added a test upload token from a test account on pypi in line with #150 .
This is my first PR here so please let me know if there's other things I should be doing. Thanks for opening this up to Hacktoberfest!