[build] Add linter configs

Author: mxmlnkn

.gitignore

@@ -2,3 +2,6 @@
 *.tar.gz
 *.egg-info
 .*
+build
+dist
+__pycache__

MANIFEST.in

@@ -39,28 +39,3 @@ content-type = "text/markdown"
 
 [tool.setuptools]
 py-modules = ["mfusepy"]
-
-[tool.flake8]
-# W503 "line break before binary operator" is directly in opposition to how black breaks lines!
-extend-ignore = ["C901", "E201", "E202", "E203", "E211", "E221", "E251", "E266", "E501", "W503"]
-max-line-length = 120
-max-complexity = 18
-select = ["B", "C", "E", "F", "W", "T4", "B9"]
-
-[tool.pylint."MESSAGES CONTROL"]
-disable = [
-    "broad-except",
-    "too-many-arguments",
-    "too-many-instance-attributes",
-    "too-many-locals",
-    "too-many-lines",
-    "too-many-positional-arguments",
-    # I don't need the style checker to bother me with missing docstrings and todos.
-    "missing-class-docstring",
-    "missing-function-docstring",
-    "missing-module-docstring",
-    "fixme",
-]
-
-[tool.mypy]
-ignore_missing_imports = true

pyproject.toml

@@ -0,0 +1,10 @@
+[flake8]
+# W503 "line break before binary operator" is directly in opposition to how black breaks lines!
+extend-ignore = C901, E201, E202, E203, E211, E221, E251, E266, E501, W503
+max-line-length = 120
+max-complexity = 18
+select = B,C,E,F,W,T4,B9
+# We need to configure the mypy.ini because the flake8-mypy's default
+# options don't properly override it, so if we don't specify it we get
+# half of the config from mypy.ini and half from flake8-mypy.
+mypy_config = .mypy.ini

tests/.flake8

@@ -0,0 +1,2 @@
+[mypy]
+ignore_missing_imports = True

tests/.mypy.ini

@@ -0,0 +1,30 @@
+[MASTER]
+# Maximum number of characters on a single line.
+max-line-length=120
+
+[MESSAGES CONTROL]
+
+# Disable the message, report, category or checker with the given id(s). You
+# can either give multiple identifiers separated by comma (,) or put this
+# option multiple times (only on the command line, not in the configuration
+# file where it should appear only once). You can also use "--disable=all" to
+# disable everything first and then reenable specific checks. For example, if
+# you want to run only the similarities checker, you can use "--disable=all
+# --enable=similarities". If you want to run only the classes checker, but have
+# no Warning level messages displayed, use "--disable=all --enable=classes
+# --disable=W".
+disable=broad-except,
+        invalid-name,
+        too-many-arguments,
+        too-many-instance-attributes,
+        too-many-locals,
+        too-many-lines,
+        too-many-positional-arguments,
+        too-many-public-methods,
+        too-few-public-methods,
+        # I don't need the style checker to bother me with missing docstrings and todos.
+        missing-class-docstring,
+        missing-function-docstring,
+        missing-module-docstring,
+        fixme,
+        unused-argument,

tests/.pylintrc

@@ -0,0 +1,53 @@
+line-length = 120
+target-version = "py37"
+
+[lint]
+select = [
+    "A", "B", "E", "F", "G", "I", "W", "N", "ASYNC", "C4", "COM", "FLY", "FURB", "ICN", "INT", "ISC", "LOG",
+    "PERF", "PIE", "PLW", "PT", "PYI", "RET", "RSE", "RUF", "SIM", "TID", "TC", "UP", "YTT"
+]
+ignore = [
+    # Preview. Complaining about spaces (aligned arguments) should be a formatter option, not a linter one!
+    # https://github.com/astral-sh/ruff/issues/2402
+    "E201", "E202", "E203", "E211", "E221", "E226", "E251", "E265", "E266", "E271",
+    "E501",     # A linter should lint, not check for line lengths!
+    "F401",     # Wants to from .version import __version__ as __version__ which clashes with pylint errors!
+    "B904",
+    "N801",     # Some class names are snake_case to match ctypes
+    "N803",     # Argument names are camelCase instead of snake_case (400+ errors).
+    "N806",     # Variable names are camelCase instead of snake_case (1100+ errors).
+    "N815",     # Class-scope variable names are camelCase instead of snake_case (10 errors).
+    "N816",     # Global variable names are camelCase instead of snake_case (73 errors).
+    "N999",     # Module names are CamelCase like the classes they provide instead of snake_case (11 errors).
+    "COM812",   # Do not force trailing commas where it makes no sense, e.g., function calls for which I'll
+                # never add more arguments.
+    "PERF203",  # Some code parts HAVE to try-except inside loops because only the element should be skipped,
+                # not the whole loop being broken out of. Furthermore, this is a useless micro-optimization,
+                # which is actually removed for Python 3.11+ which introduces zero-cost exceptions.
+    "PLW0603",  # Cannot live without global statements, especially for subprocessing.
+    "PLW2901",  # Only false positives for constructs such as for data in ...: data = data.clean(); process(data)
+    "RET504",   # https://github.com/astral-sh/ruff/issues/17292#issuecomment-3039232890
+    "RUF001",   # Only false positives from tests with Unicode characters.
+    "RUF039",   # https://github.com/astral-sh/ruff/issues/18795
+    "RUF056",   # "options.get('disableUnionMount', False):" Wants me to change False to None,
+                # but it makes no sense semantically (expecting a bool)
+    "RUF100",   # BUG: removes necessary noqa: E402 in tests!
+    "RUF012",   # Mutable class attributes should be annotated with `typing.ClassVar`, but _fields_ is not mutable.
+
+    "SIM102",
+    "SIM105",
+    "SIM115",   # Too many locations where I do not know how to use context managers for files,
+                # which are stored as object members for example!
+    "TC006",    # Wants to quote cast("IO[bytes]", ...) I don't agree with this style choice.
+    "S101", "S110", "S105", "S311", "S324", "S603", "S607", "S608"
+
+    # Bug: SIM118 removes the keys() from row.keys(), which is an sqlite3.Row not a Dict!
+]
+# Allow fix for all enabled rules (when `--fix`) is provided.
+fixable = ["ALL"]
+unfixable = []
+
+[format]
+line-ending = "lf"
+quote-style = "preserve"
+

tests/.ruff.toml

@@ -0,0 +1,11 @@
+#!/usr/bin/env bash
+
+# Will print warnings for each folder because find tries to descend into it after finding those.
+# It may be confusing but it is a nice verbose output to list deleted folders.
+find . -type d '(' \
+    -name '*.egg-info' -or -name '*.mypy_cache' -or -name '__pycache__' -or '.ruff_cache' -or \
+    -name '*.pytest_cache' -or -name '*.pytype' -or -name 'dist' -or -name 'build' \
+')' -exec rm -rf {} ';'
+'rm' -f httpd-ruby-webrick.log ratarmount.stderr.log.tmp ratarmount.stdout.log.tmp
+
+for service in httpd ipfs pyftpdlib wsgidav; do pkill -f "$service"; done