diff options
author | Pierre Sassoulas <pierre.sassoulas@gmail.com> | 2022-12-28 17:17:03 +0100 |
---|---|---|
committer | Daniƫl van Noord <13665637+DanielNoord@users.noreply.github.com> | 2022-12-28 17:35:57 +0100 |
commit | 8912838a27f3391d78e11ed32ce40b58074d8d22 (patch) | |
tree | cfbf2d47086a7788188f6dcbe539253f8da1a2ce /pylint/checkers/refactoring/refactoring_checker.py | |
parent | 160add5cbcdd9fdf7415fadef768c913e3ab114a (diff) | |
download | pylint-git-8912838a27f3391d78e11ed32ce40b58074d8d22.tar.gz |
[consider-using-sys-exit] Better rationale and added confidence
Diffstat (limited to 'pylint/checkers/refactoring/refactoring_checker.py')
-rw-r--r-- | pylint/checkers/refactoring/refactoring_checker.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/pylint/checkers/refactoring/refactoring_checker.py b/pylint/checkers/refactoring/refactoring_checker.py index ed8f8e052..f20795e29 100644 --- a/pylint/checkers/refactoring/refactoring_checker.py +++ b/pylint/checkers/refactoring/refactoring_checker.py @@ -405,9 +405,10 @@ class RefactoringChecker(checkers.BaseTokenChecker): "It is faster and simpler.", ), "R1722": ( - "Consider using sys.exit()", + "Consider using 'sys.exit' instead", "consider-using-sys-exit", - "Instead of using exit() or quit(), consider using the sys.exit().", + "Contrary to 'exit()' or 'quit()', 'sys.exit' does not rely on the " + "site module being available (as the 'sys' module is always available).", ), "R1723": ( 'Unnecessary "%s" after "break", %s', @@ -1121,7 +1122,7 @@ class RefactoringChecker(checkers.BaseTokenChecker): node.root() ): return - self.add_message("consider-using-sys-exit", node=node) + self.add_message("consider-using-sys-exit", node=node, confidence=HIGH) def _check_super_with_arguments(self, node: nodes.Call) -> None: if not isinstance(node.func, nodes.Name) or node.func.name != "super": |