-
Notifications
You must be signed in to change notification settings - Fork 10
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
Overhauled packaging and testing #13
Conversation
Thanks for the Contributions @agronholm 🥳 |
I mistakenly included Python 3.8 in the test workflow; you'll need to delete that. The rest fail because they seem to require a Windows-specific module. |
One more thing: you have to configure trusted publishing from the PyPI side too: https://docs.pypi.org/trusted-publishers/adding-a-publisher/ |
Yeah, I configured that when pushed the v1.0.2 tag. |
I'll remove Python 3.8 from the test workflow, will need to look into the Windows-specific module one though. |
It seems like you're using |
I see, we shifted from |
This is the commit that introduced |
got it thanks! will fix this soon |
One more question: you removed the |
No, they'd just install it in editable mode: |
setup.py
topyproject.toml
requirements.txt
topyproject.toml
(under thetest
extra)tkreload/__init__.py
as no longer needed (no tools uses these)README.MD
files toREADME.md
for consistency