Skip to content

Bump Pylint to 2.17 and flake8 to 6.0 #359

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
bebound merged 30 commits into from
Jul 14, 2023
Merged

Bump Pylint to 2.17 and flake8 to 6.0 #359

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
ac7c673
Update
bebound Feb 6, 2023
44c36ee
Minor fix
bebound Feb 6, 2023
d4070e8
Enable no_self_use
bebound Feb 6, 2023
1928f29
Minor fix
bebound Feb 6, 2023
c60042d
Minor fix
bebound Feb 6, 2023
3a22e88
Minor fix
bebound Feb 6, 2023
87cb41c
Minor fix
bebound Feb 6, 2023
2fd99af
Fix flake8
bebound Feb 6, 2023
998a0bc
Update setup.py
dciborow Mar 10, 2023
4cd84e7
Update cli_pylintrc
dciborow Mar 10, 2023
7a2c4fe
Update ext_pylintrc
dciborow Mar 10, 2023
a4bb458
Merge branch 'dev' into dciborow/pylint
dciborow Mar 10, 2023
0fa543d
Merge branch 'dev' into dciborow/pylint
dciborow May 23, 2023
a071b95
Update .pylintrc
dciborow May 24, 2023
1028936
Apply suggestions from code review
dciborow May 24, 2023
d207de8
Update setup.py
dciborow May 24, 2023
23cb1db
Update azdev/operations/testtool/tests/test_profile_context.py
dciborow May 24, 2023
ec84363
Apply suggestions from code review
dciborow May 24, 2023
513539f
Update ext_pylintrc
dciborow May 24, 2023
664c6c8
Update cli_pylintrc
dciborow May 24, 2023
074f8aa
Update .pylintrc
dciborow May 24, 2023
510f0c1
Apply suggestions from code review
dciborow May 24, 2023
f07822b
Update azdev/operations/help/__init__.py
bebound Jun 30, 2023
a4623ce
Update setup.py
bebound Jun 30, 2023
ae28381
Merge branch 'dev' into dciborow/pylint
bebound Jun 30, 2023
53f9b3d
Ignore missing-timeout
bebound Jun 30, 2023
4d2e2a5
Merge branch 'dev' into dciborow/pylint
bebound Jul 11, 2023
0e60fba
Minor fix
bebound Jul 12, 2023
f80f150
Enable deprecated-module
bebound Jul 12, 2023
08e6383
Remove useless config
bebound Jul 13, 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
Prev Previous commit
Next Next commit
Fix flake8
  • Loading branch information
@bebound
bebound committed Feb 6, 2023
commit 2fd99af251f1547a6be61420d4d592c175ec414c
20 changes: 13 additions & 7 deletions azdev/config/cli.flake8
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,23 @@
max-line-length = 120
max-complexity = 10
ignore =
E501, # line too long, it is covered by pylint
E722, # bare except, bad practice, to be removed in the future
F401, # imported but unused, too many violations, to be removed in the future
F811, # redefinition of unused, to be removed in the future
C901 # code flow is too complex, too many violations, to be removed in the future
W504 # line break after binary operator effect on readability is subjective
# line too long, it is covered by pylint
E501,
Comment on lines +5 to +6
Copy link
Member

@jiasli jiasli Jul 11, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason for reformatting?

Copy link
Contributor

@bebound bebound Jul 11, 2023 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because flake8 upgrade to 6.0. Flake8 6.0 does not support inline comments for any of the keys, so comments should be put in newline.
Use this to prevent ValueError: Error code '#' supplied to 'ignore' option does not match '^[A-Z]{1,3}[0-9]{0,3}$'

Related issue: PyCQA/flake8#1750 Azure/azure-cli#25370

Copy link
Contributor

@bebound bebound Jul 11, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When pylint==2.11.1, it installs mccabe 0.6.1. flake8 also requires mccabe, and only 4.0.1 meet the version constraint.

  - pylint [required: ==2.11.1, installed: 2.11.1]
    - astroid [required: >=2.8.0,<2.9, installed: 2.8.6]
      - lazy-object-proxy [required: >=1.4.0, installed: 1.9.0]
      - setuptools [required: >=20.0, installed: 63.2.0]
      - wrapt [required: >=1.11,<1.14, installed: 1.13.3]
    - isort [required: >=4.2.5,<6, installed: 5.12.0]
    - mccabe [required: >=0.6,<0.7, installed: 0.6.1]
    - platformdirs [required: >=2.2.0, installed: 2.6.2]
    - toml [required: >=0.7.1, installed: 0.10.2]
 
- flake8 [required: Any, installed: 4.0.1]
    - mccabe [required: >=0.6.0,<0.7.0, installed: 0.6.1]
    - pycodestyle [required: >=2.8.0,<2.9.0, installed: 2.8.0]
    - pyflakes [required: >=2.4.0,<2.5.0, installed: 2.4.0]

After upgrading PyLint 2.7, mccabe becomes [required: >=0.6,<0.8, installed: 0.7.0], and flake8 becomes 6.0.0.

Copy link
Member

@jiasli jiasli Jul 13, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this issue still exist in pylint 2.17.4? https://pypi.org/project/pylint/

Copy link
Contributor

@bebound bebound Jul 13, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When run azdev style, it uses pylintrc and .flake8 in CLI/Extension repo folder. If the files do not exist, it use cli.xxxx or ext.xxx in azdev config folder.
CLI's .flake8 is updated in Azure/azure-cli#25370.

# bare except, bad practice, to be removed in the future
E722,
# imported but unused, too many violations, to be removed in the future
F401,
# redefinition of unused, to be removed in the future
F811,
# code flow is too complex, too many violations, to be removed in the future
C901,
# line break after binary operator effect on readability is subjective
W504
exclude =
azure_cli_bdist_wheel.py
build
tools
scripts
doc
build_scripts
*/grammar/
*/grammar/
18 changes: 12 additions & 6 deletions azdev/config/ext.flake8
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,18 @@
max-line-length = 120
max-complexity = 10
ignore =
E501, # line too long, it is covered by pylint
E722, # bare except, bad practice, to be removed in the future
F401, # imported but unused, too many violations, to be removed in the future
F811, # redefinition of unused, to be removed in the future
C901 # code flow is too complex, too many violations, to be removed in the future
W504 # line break after binary operator effect on readability is subjective
# line too long, it is covered by pylint
E501,
# bare except, bad practice, to be removed in the future
E722,
# imported but unused, too many violations, to be removed in the future
F401,
# redefinition of unused, to be removed in the future
F811,
# code flow is too complex, too many violations, to be removed in the future
C901,
# line break after binary operator effect on readability is subjective
W504
exclude =
*/vendored_sdks
docs
Expand Down