def _check_boolean_arguments(self, node: ast.Call) -> None: for arg in node.args: if isinstance(arg, ast.NameConstant): # We do not check for `None` values here: if arg.value is True or arg.value is False: self.add_violation( BooleanPositionalArgumentViolation(node), ) break
def _check_boolean_arguments(self, node: ast.Call) -> None: # Calls with single boolean argument are allowed if len(node.args) == 1 and not node.keywords: return for arg in node.args: if isinstance(arg, ast.NameConstant): # We do not check for `None` values here: if arg.value is True or arg.value is False: self.add_violation( BooleanPositionalArgumentViolation( arg, text=str(arg.value), ), )
def _check_boolean_arguments(self, node: ast.Call) -> None: if len(node.args) == 1 and not node.keywords: return # Calls with single boolean argument are allowed for arg in node.args: if not isinstance(arg, ast.NameConstant): continue is_ignored = self._is_call_ignored(node) # We do not check for `None` values here: if not is_ignored and arg.value in {True, False}: self.add_violation( BooleanPositionalArgumentViolation( arg, text=str(arg.value), ), )