From 5b58a3f52f8347f2d3e439196514dc1a41292101 Mon Sep 17 00:00:00 2001 From: l0lawrence Date: Tue, 16 Jul 2024 09:36:23 -0700 Subject: [PATCH 1/4] pylintrc updates for -next-next pylint --- packages/autorest.python/pylintrc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/autorest.python/pylintrc b/packages/autorest.python/pylintrc index 558c7fb6cfb..53e9cbbfa50 100644 --- a/packages/autorest.python/pylintrc +++ b/packages/autorest.python/pylintrc @@ -7,12 +7,14 @@ reports=no ignore=_generated,samples,examples,test,tests,doc,.tox,generated_samples [MESSAGES CONTROL] + +enable=useless-disable # For all codes, run 'pylint --list-msgs' or go to 'https://pylint.readthedocs.io/en/latest/reference_guide/features.html' # locally-disabled: Warning locally suppressed using disable-msg # cyclic-import: because of https://github.com/PyCQA/pylint/issues/850 # too-many-arguments: Due to the nature of the CLI many commands have large arguments set which reflect in large arguments set in corresponding methods. # Let's black deal with bad-continuation -disable=useless-object-inheritance,missing-docstring,locally-disabled,fixme,cyclic-import,too-many-arguments,invalid-name,duplicate-code,too-few-public-methods,consider-using-f-string,super-with-arguments,redefined-builtin,import-outside-toplevel,client-suffix-needed,unnecessary-dunder-call,unnecessary-ellipsis,disallowed-name +disable=useless-object-inheritance,missing-docstring,locally-disabled,fixme,cyclic-import,too-many-arguments,invalid-name,duplicate-code,too-few-public-methods,consider-using-f-string,super-with-arguments,redefined-builtin,import-outside-toplevel,client-suffix-needed,unnecessary-dunder-call,unnecessary-ellipsis,disallowed-name,consider-using-max-builtin,possibly-used-before-assignment [FORMAT] max-line-length=120 From 63e1c473d6c86821bddaf4c40acfa16d0f778cd5 Mon Sep 17 00:00:00 2001 From: l0lawrence Date: Tue, 16 Jul 2024 09:37:10 -0700 Subject: [PATCH 2/4] remove --- packages/autorest.python/pylintrc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/autorest.python/pylintrc b/packages/autorest.python/pylintrc index 53e9cbbfa50..eb3346264e0 100644 --- a/packages/autorest.python/pylintrc +++ b/packages/autorest.python/pylintrc @@ -14,7 +14,7 @@ enable=useless-disable # cyclic-import: because of https://github.com/PyCQA/pylint/issues/850 # too-many-arguments: Due to the nature of the CLI many commands have large arguments set which reflect in large arguments set in corresponding methods. # Let's black deal with bad-continuation -disable=useless-object-inheritance,missing-docstring,locally-disabled,fixme,cyclic-import,too-many-arguments,invalid-name,duplicate-code,too-few-public-methods,consider-using-f-string,super-with-arguments,redefined-builtin,import-outside-toplevel,client-suffix-needed,unnecessary-dunder-call,unnecessary-ellipsis,disallowed-name,consider-using-max-builtin,possibly-used-before-assignment +disable=useless-object-inheritance,missing-docstring,locally-disabled,fixme,cyclic-import,too-many-arguments,invalid-name,duplicate-code,too-few-public-methods,consider-using-f-string,super-with-arguments,redefined-builtin,import-outside-toplevel,client-suffix-needed,unnecessary-dunder-call,unnecessary-ellipsis,disallowed-name,consider-using-max-builtin [FORMAT] max-line-length=120 From 19a291e52a30724c8d455cb56e3c8fb3eae89087 Mon Sep 17 00:00:00 2001 From: l0lawrence Date: Tue, 16 Jul 2024 10:35:05 -0700 Subject: [PATCH 3/4] typo --- packages/autorest.python/pylintrc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/autorest.python/pylintrc b/packages/autorest.python/pylintrc index eb3346264e0..9d0386db200 100644 --- a/packages/autorest.python/pylintrc +++ b/packages/autorest.python/pylintrc @@ -8,7 +8,9 @@ ignore=_generated,samples,examples,test,tests,doc,.tox,generated_samples [MESSAGES CONTROL] -enable=useless-disable +# Add enable for useless disables +enable=useless-suppression + # For all codes, run 'pylint --list-msgs' or go to 'https://pylint.readthedocs.io/en/latest/reference_guide/features.html' # locally-disabled: Warning locally suppressed using disable-msg # cyclic-import: because of https://github.com/PyCQA/pylint/issues/850 From 1a4c69622054e3895f435758621a3891e3601588 Mon Sep 17 00:00:00 2001 From: l0lawrence Date: Wed, 17 Jul 2024 12:40:04 -0700 Subject: [PATCH 4/4] add summary --- .chronus/changes/pylintUpdates-2024-6-17-12-39-48.md | 7 +++++++ 1 file changed, 7 insertions(+) create mode 100644 .chronus/changes/pylintUpdates-2024-6-17-12-39-48.md diff --git a/.chronus/changes/pylintUpdates-2024-6-17-12-39-48.md b/.chronus/changes/pylintUpdates-2024-6-17-12-39-48.md new file mode 100644 index 00000000000..4c0d53cc642 --- /dev/null +++ b/.chronus/changes/pylintUpdates-2024-6-17-12-39-48.md @@ -0,0 +1,7 @@ +--- +changeKind: internal +packages: + - "@autorest/python" +--- + +Updating pylintrc rules \ No newline at end of file