-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Remove pylint constraint #36169
Remove pylint constraint #36169
Conversation
Thanks for the pull request, @UsamaSadiq! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review.
|
9921be6
to
dd408bb
Compare
75e8d12
to
c26e7cd
Compare
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. |
1 similar comment
2U Release Notice: This PR has been deployed to the edX production environment. |
Description
pylint
constraint and fixing the failing quality checksTODO
pylintrc_tweaks
will be removed and warnings will be fixed in a follow up PR since a lot of files will need to be refactored for this change.