-
Notifications
You must be signed in to change notification settings - Fork 53
Update CI #93
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
Update CI #93
Conversation
…S. Also run tests under Python 3.13 as well.
Looks like we're running into issue: https://sourceforge.net/p/docutils/bugs/483/ I'll simply remove the problematic dev-dependency (an install it manually from the workflow) for now so we can have a green CI |
Looks like I'm running into this issue specifically on windows/python3.8: pypa/setuptools#4483
I also ran into pypa/setuptools#4501 along the way. Specifically on Python3.8/Windows runner... other are fine |
Ok seems like I can't make it run under Windows/Py3.8. |
Maybe you want to try this again, now that it's not poetry based? |
with: | ||
python-version: 3.8 | ||
python-version: 3.12 |
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-version: 3.12 | |
python-version: 3.x |
If you want to minimize maintenance
Co-authored-by: Cristian Le <[email protected]>
@@ -40,7 +40,7 @@ test = [ | |||
"pytest", | |||
] | |||
docs = [ | |||
"pydoctor >= 22.3.0", | |||
"pydoctor >= 25.4.0", |
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.
My only remaining comment is about this change? Is it needed?
Update Github workflow to use "latest" versions of the three tested OS. Also run tests under Python 3.13 as well.