-
-
Notifications
You must be signed in to change notification settings - Fork 393
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
build: configure type checkers to run on min supported version #2474
Conversation
This PR modifies our type-checking to run on the minimum supported version, instead of the latest. Rationale: - Reduce incidence of backward incompatible changes. - Improve local development workflow - prior to this change, local development requires 2 python environments, 1 for 3.8 for dev/running tests, and one for 3.12 to run the type checkers. After this change developers will only have to run a single python environment for running both tests and type-checks.
3d161de
to
7329c03
Compare
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.
Maybe can suggest RTX as well
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.
Are these required for end users or just available? Sorry I haven't caught up to this new change
Sorry @JacobCoffee can you clarify the question please? |
In |
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.
Last final nit :)
Co-authored-by: Nikita Sobolev <[email protected]>
Kudos, SonarCloud Quality Gate passed! |
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.
Thank you! Great work!
Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/2474 |
Pull Request Checklist
Description
This PR modifies our type-checking to run on the minimum supported version, instead of the latest.
Rationale:
Close Issue(s)