Skip to content
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.

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

flake8: fix warnings E111, E123, E126, E128, E201, E202, E203, E225, E226, E231, E241, E251, E261, E265, E266, E305, E305, E402, E501, E502, E703, E722 #5599

Open
ligurio opened this issue Dec 3, 2020 · 0 comments
Labels
good first issue Good for newcomers qa Issues related to tests or testing subsystem

Comments

@ligurio
Copy link
Member

ligurio commented Dec 3, 2020

Bug description:

  • E111: indentation is not a multiple of four
  • E123: closing bracket does not match indentation of opening bracket's line
  • E126: continuation line over-indented for hanging indent
  • E128: continuation line under-indented for visual indent
  • E201: whitespace after '('
  • E202: whitespace before ')'
  • E203: whitespace before ':'
  • E225: missing whitespace around operator
  • E226: missing whitespace around arithmetic operator
  • E231: missing whitespace after ',', ';' or ':'
  • E241: multiple spaces after ','
  • E251: unexpected spaces around keyword / parameter equals
  • E261: at least two spaces before inline comment
  • E265: block comment should start with '#'
  • E266: too many leading '#' for block comment
  • E302: expected 2 blank lines, found 0
  • E305: expected 2 blank lines after end of function or class
  • E402: module level import not at top of file
  • E501: line too long (82 > 79 characters)
  • E502: the backslash is redundant between brackets
  • E703: statement ends with a semicolon
  • E722: do not use bare except, specify exception instead
@ligurio ligurio added qa Issues related to tests or testing subsystem good first issue Good for newcomers labels Dec 3, 2020
ligurio added a commit that referenced this issue Dec 3, 2020
- add target 'make flake8'
- add initial flake8 config with supressed warnings

#5599
#5598
#5597
ligurio added a commit that referenced this issue Dec 3, 2020
flake8 configuration file supress warnings, that can be fixed later.
Supressed warnings can be splitted for three categories and for each of them
separate issue exists:

- Fix warnings Exx: #5599
- Fix warnings Fxx: #5598
- Fix warnings Wxx: #5597
ligurio added a commit that referenced this issue Dec 3, 2020
flake8 configuration file supress warnings, that can be fixed later.
Supressed warnings can be splitted for three categories and for each of them
separate issue exists:

- Fix warnings Exx: #5599
- Fix warnings Fxx: #5598
- Fix warnings Wxx: #5597
ligurio added a commit that referenced this issue Dec 6, 2020
flake8 configuration file supress warnings, that can be fixed later.
Supressed warnings can be splitted for three categories and for each of them
separate issue exists:

- Fix warnings Exx: #5599
- Fix warnings Fxx: #5598
- Fix warnings Wxx: #5597
ligurio added a commit that referenced this issue Dec 6, 2020
flake8 configuration file supress warnings, that can be fixed later.
Supressed warnings can be splitted for three categories and for each of them
separate issue exists:

- Fix warnings Exx: #5599
- Fix warnings Fxx: #5598
- Fix warnings Wxx: #5597
sdudoladov pushed a commit to sdudoladov/tarantool that referenced this issue Mar 19, 2021
This patch introduces the .flake8 configuration file
and fixes the following flake8 warnings:
* E201: whitespace after  '('
* E202: whitespace before ')'
* E225: missing whitespace around operator

to improve code quality for tests written in Python

Needed for: tarantool#5599
sdudoladov pushed a commit to sdudoladov/tarantool that referenced this issue Mar 19, 2021
This patch introduces the .flake8 configuration file
and fixes the following flake8 warnings:
* E201: whitespace after  '('
* E202: whitespace before ')'
* E225: missing whitespace around operator

to improve code quality for tests written in Python

Needed for: tarantool#5599
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers qa Issues related to tests or testing subsystem
Projects
None yet
Development

No branches or pull requests

1 participant