From 78a346124832026038b21ddb2502a15219c0d49d Mon Sep 17 00:00:00 2001 From: Brad Warren Date: Tue, 23 Nov 2021 09:15:53 -0800 Subject: [PATCH] disable the refactoring checker --- .pylintrc | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.pylintrc b/.pylintrc index ccd660239..84b033a7e 100644 --- a/.pylintrc +++ b/.pylintrc @@ -72,7 +72,12 @@ extension-pkg-whitelist=pywintypes,win32api,win32file,win32security # (unspecified encoding makes the open function use the default encoding of the system) # than a clear flaw on which a check should be enforced. Anyway the project does # not need to enforce encoding on files so we disable this check. -disable=fixme,locally-disabled,locally-enabled,bad-continuation,no-self-use,invalid-name,cyclic-import,duplicate-code,design,import-outside-toplevel,useless-object-inheritance,unsubscriptable-object,no-value-for-parameter,no-else-return,no-else-raise,no-else-break,no-else-continue,raise-missing-from,wrong-import-order,unspecified-encoding +# 7) pylint's refactoring checker makes suggestions on how code could be +# structured differently. These checks are very opinionated and have caused +# pylint to fail on Certbot when it's upgraded so let's disable this checker +# entirely. You can see a list of the things the checker considers at +# https://pylint.pycqa.org/en/v2.11.1/technical_reference/features.html#refactoring-checker. +disable=fixme,locally-disabled,locally-enabled,bad-continuation,no-self-use,invalid-name,cyclic-import,duplicate-code,design,import-outside-toplevel,useless-object-inheritance,unsubscriptable-object,no-value-for-parameter,no-else-return,no-else-raise,no-else-break,no-else-continue,raise-missing-from,wrong-import-order,unspecified-encoding,refactoring [REPORTS]