-
Notifications
You must be signed in to change notification settings - Fork 4.2k
chore: Upgrade Python requirements #36837
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
Conversation
|
List of packages in the PR without any issue.
|
|
These Packages need manual review..
|
When it was located at the root of the project, it was sometimes not
getting loaded correctly since the root of the project is not a python
module, and producing the following error:
```
Command line or configuration file:1:0: E0013: Plugin 'pylint_django_settings' is impossible to load, is it installed ? ('No module named 'pylint_django_settings'') (bad-plugin-value)
```
This led all the pylint tests to fail.
This started happening more as we updated other dependencies via make
upgrade for some reason and led to inconsistent builds. The move should
hopefully make the loading reliable and consistent.
5d38baa to
49158e7
Compare
It was included in both the openedx-1 and openedx-2 lint suites.
|
Closing obsolete PR. |
Django stubs by default doesn't pin mypy but has an extras that does properly pin it to versions that are compatible. For us it doesn't make sense to not use this extra as we will always need a version of Mypy that works with djnago-stubs.
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
|
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
|
2U Release Notice: This PR has been deployed to the edX production environment. |
Python requirements update. Please review the changelogs for the upgraded packages.
Deleted obsolete pull_requests:
#36798