Use the Flake8 tool and set things up so that it scans our code automatically
as we send PRs and provides feedback on the PR review screen.
Install Flake8.
Configure Flake8 to set the maximum-line-length to be 120, via the line:
pipenv run flake8 --max-line-length 120
In Github, setup branch-protection of the main branch, by asserting that
every PR to the main branch must be approved by the Flake8 tool.
NOTE: That means, no one would be able to push changes to the main
branch without requesting an official PR.
Do it like so:
Go to the repository's “settings” and select “branches”,
then click the “add rule” button.
Write “main” in the “Branch name pattern” field.
Select “Require status checks to pass before merging”.
In the search field that now appears, write the name of our CI test from the .yml file:
“Check code with Flake8”, and select the same name from the dropdown.
Click the “Create” button at the bottom of this page to create the rule.
Use the Flake8 tool and set things up so that it scans our code automatically as we send PRs and provides feedback on the PR review screen.
In Github, setup branch-protection of the
mainbranch, by asserting that every PR to themainbranch must be approved by the Flake8 tool.Do it like so:
.ymlfile: “Check code with Flake8”, and select the same name from the dropdown.