Apply Pylint on CI on modified files only
Context
Adding a linter in an existing project is a good thing.
Currently, Pylint (and black and Flake8) are only used as information in the CI pipeline (allow_failure: true
).
This means that whatever the lint
stage outputs in the CI, pipeline will be validated.
It may results in developers not paying attention to this stage for merge request.
Improvements
As the current codebase is not fulfilling the linter specs yet, we want to add a way to make the code cleaner step by step. One way would be to compare the codebase before and after changes and validate changes (in a merge request for lint stage) if they improve the code quality.
Edited by Tom FRANCOIS