Skip to:
Since we use zuul with flake8/black, other linters (pycodestyle/pylint) are not really used
To avoid noise, we should just wipe them from all repos
Workflow
create PR on only one repo and ask to someone to review
create all others PR with auto-merge label
If a PR is different from the first one, do not use auto-merge label and ask for a review
Note: Also wipe code coverage logic for pytest, because it is not used anymore
It was used by our CI jenkins, but since zuul migration, they are not read/exposed anymore
Metric to have code coverage is in discussion. So I’ll exclude it from the scope of this PR for now
All PRs are merged into master, the review is only for knowledge transfer
Since we use zuul with flake8/black, other linters (pycodestyle/pylint) are not really used
To avoid noise, we should just wipe them from all repos
Workflow
create PR on only one repo and ask to someone to review
create all others PR with auto-merge label
If a PR is different from the first one, do not use auto-merge label and ask for a review
Note: Also wipe code coverage logic for pytest, because it is not used anymore
It was used by our CI jenkins, but since zuul migration, they are not read/exposed anymore