-
Notifications
You must be signed in to change notification settings - Fork 24
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
Drop Python 3.8 support and add Python 3.12 #302
Conversation
poetry.lock
Outdated
@@ -1,4 +1,4 @@ | |||
# This file is automatically @generated by Poetry 1.8.3 and should not be changed by hand. | |||
# This file is automatically @generated by Poetry 1.6.1 and should not be changed by hand. |
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.
What necessitated regenerating/updating the poetry.lock with new dependency versions?
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.
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.
Not a bad thing to move poetry versions forward. Any harm that I'm not aware of on this?
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 like this is moving the poetry
version itself backwards, which seems wrong. Just updating the min Python version shouldn't require re-locking all other dependencies (poetry lock --no-update
would suffice); I'm not opposed to it in principle but it might be better done as a separate PR.
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, I see what you mean, I shouldn't move back
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 have regenerated with the latest poetry. I agree that a separate PR could have been better, but I would keep it together to simplify it for now.
No description provided.