Hi everyone,
Under the implementation of OEP-34: Lint Amnesty, following changes have been made in edx-platform quality tests through the PR:
Changes :
-
lint-amnesty has been applied on all the existing
pylintviolations and thepylint warning thresholdshave been removed in BOM-2477 . - Quality build now fails in case of any
pylint warningwhich can be fixed by following steps described in ways to fix pylint violations. -
diff-qualityhad been added for maintaining Python Code Style Quality. BOM-2543 removed its implementation from the quality build to make sure all the warnings are reported correctly. For details see Differential Linting example.
Impact on quality builds:
- Previously, a quality build could be made to pass by increasing the warning thresholds instead of actually fixing the warnings. Now, either fixing/suppressing the warning will be necessary for the build to pass.