Skip to content

Enable four more flake8 rules #7811

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.

Sign up for GitHub

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

Merged
DaanHoogland merged 2 commits into from
Oct 10, 2023
Merged

Enable four more flake8 rules #7811

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b9206f7
Enable four more `flake8` rules
jbampton Aug 3, 2023
3c68862
Merge branch 'main' into enable-more-flake8-rules
jbampton Aug 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion .github/linters/.flake8
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,12 @@
# specific language governing permissions and limitations
# under the License.

# E112 Expected an indented block
# E113 Unexpected indentation
# E133 Closing bracket is missing indentation
# E223 Tab before operator
# E224 Tab after operator
# E227 Missing whitespace around bitwise or shift operator
# E242 Tab after ','
# E273 Tab after keyword
# E274 Tab before keyword
Expand All @@ -30,4 +34,7 @@
# W391 Blank line at end of file

[flake8]
select = E223,E224,E242,E273,E274,E742,E743,E901,E902,W291,W292,W293,W391
exclude =
.git,
venv
select = E112,E113,E133,E223,E224,E227,E242,E273,E274,E742,E743,E901,E902,W291,W292,W293,W391