-
Notifications
You must be signed in to change notification settings - Fork 1
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
Support for Python 3.12 #35
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.
See comments. poetry.lock
needs to be regenerated
.github/workflows/main.yml
Outdated
@@ -23,7 +23,7 @@ jobs: | |||
runs-on: ubuntu-22.04 | |||
strategy: | |||
matrix: | |||
python_version: ['3.8', '3.9', '3.10', '3.11'] | |||
python_version: ['3.9', '3.10', '3.11', '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.
Is there a reason, you removed 3.8 from the tests?
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.
Yeah I should put that back, with some repos careful to include minimal set due to time, but this is pretty quick.
pyproject.toml
Outdated
python = ">=3.8.1,<3.13" | ||
dcicutils = "^8.13.3" | ||
docutils = ">=0.10,<0.17," | ||
tibanna-ff = "3.4.2.1b2" |
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.
Adjust version upon release of tibanna-ff
pyproject.toml
Outdated
tibanna-ff = ">=3.2.0" | ||
python = ">=3.8.1,<3.13" | ||
dcicutils = "^8.13.3" | ||
docutils = ">=0.10,<0.17," |
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.
Why do we need this now?
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.
Removed; was artifact of some troubleshooting of some problem; not needed there though.
python = ">=3.8,<3.12" | ||
dcicutils = "^8.0.0" | ||
tibanna-ff = ">=3.2.0" | ||
python = ">=3.8.1,<3.13" |
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.
I would not exclude 3.8.0 here if there is no good reason. Technically that would be a breaking change.
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.
Okay, will have to make dcicutils be >=3.8 too which requires downgrading flake8, etc; I can see how that goes.
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.
Oh, I see. You can leave 3.8.1 then. It's just us using it. Should be fine.
No description provided.