Skip to content

Commit

Permalink
DEV: remove mccabe code complexity check from the ruff config
Browse files Browse the repository at this point in the history
It has given issues a couple of times that take time to resolve,
and the complaints it emits seem unjustified at least in some
cases. So remove it - relying on code review is fine here.
  • Loading branch information
rgommers authored and dnicolodi committed Sep 12, 2023
1 parent 0397288 commit 1e79b26
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
2 changes: 1 addition & 1 deletion mesonpy/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -603,7 +603,7 @@ def _string_or_strings(value: Any, name: str) -> List[str]:
class Project():
"""Meson project wrapper to generate Python artifacts."""

def __init__( # noqa: C901
def __init__(
self,
source_dir: Path,
build_dir: Path,
Expand Down
5 changes: 0 additions & 5 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,6 @@ extend-ignore = [
select = [
'B', # flake8-bugbear
'C4', # flake8-comprehensions
'C9', # mccabe
'E', # pycodestyle
'F', # pyflakes
'W', # pycodestyle
Expand All @@ -92,10 +91,6 @@ exclude = [
'docs/conf.py',
]

[tool.ruff.mccabe]
max-complexity = 12


[tool.isort]
lines_between_types = 1
lines_after_imports = 2
Expand Down

0 comments on commit 1e79b26

Please sign in to comment.