-
Notifications
You must be signed in to change notification settings - Fork 118
Add Django 5.2 support #461
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
base: master
Are you sure you want to change the base?
Conversation
Pull Request Test Coverage Report for Build 16200018383Details
💛 - Coveralls |
Look like CI on main became broken and pre-commit too. Change LGTM, otherwise. (You can make the branch writtable by contributors so we can handle little things for you) |
@Pierre-Sassoulas Thank you for the review! I’d like to try fixing the CI errors myself first. If I need any help, I’ll reach out here. If I’m not able to resolve the issues, I’ll make my branch available for contributors. |
@Pierre-Sassoulas It looks like the test_latest CI job is failing due to an upstream dependency conflict between the main branches of pylint and astroid. ![]()
|
The fix is to rely on bleeding edge pylint only and remove the dependency to bleeding edge astroid. Bleeding edge pylint should control its dependencies. And having bleeding edge pylint be always compatible with bleeding edge astroid is something pylint is not able to guarantee in a reasonable amount of time for a reasonable amount of maintainer time. (And impossible to always be true when there's breaking changes in astroid) |
#460