Switch to ruff for linting (#261)

* Switch to Ruff for linting

* Remove flak8, pep8, isort, prospector config

Use ruff instead

Author: davidfischer

.ruff.toml

@@ -0,0 +1,38 @@
+# https://docs.astral.sh/ruff/configuration/
+extend-include = ["*.ipynb"]
+
+exclude = [
+  "manage.py",
+  "docs",
+  "settings/",
+  "common/",
+  "readthedocs/rtd_tests/",
+  "readthedocs/templates/",
+  "**/tests/*.py",
+  "**/test_*.py",
+  "**/tests.py",
+  "**/conftest.py",
+]
+
+# Same as Black.
+line-length = 100
+indent-width = 4
+
+# Assume Python 3.12
+target-version = "py312"
+
+[lint]
+# Enable:
+#  Pyflakes (`F`)
+#  pycodestyle (`E`) error codes.
+#  isort (`I`) import sorting
+select = ["E4", "E7", "E9", "F", "I"]
+
+[lint.isort]
+# https://docs.astral.sh/ruff/settings/#lintisort
+force-single-line = true
+case-sensitive = false
+lines-after-imports = 2
+
+# https://docs.astral.sh/ruff/settings/#lint_isort_known-first-party
+known-first-party = ["readthedocs", "readthedocsinc", "readthedocsext"]

flake8

@@ -38,81 +38,29 @@ repos:
     - id: mixed-line-ending
       args: ['--fix=lf']
 
-- repo: https://github.com/PyCQA/autoflake
-  rev: v2.2.0
-  hooks:
-    - id: autoflake
-      args: ['--in-place', '--remove-all-unused-imports', '--remove-unused-variable']
-
-# TODO: re-enable it and use `--range` to select the modified lines
-# - repo: https://github.com/myint/docformatter.git
-#   rev: v1.4
-#   hooks:
-#     - id: docformatter
-#       args: ['--in-place', '--wrap-summaries=80', '--wrap-descriptions=80', '--pre-summary-newline']
-
 - repo: https://github.com/adamchainz/django-upgrade
   rev: "1.14.1"
   hooks:
   - id: django-upgrade
     args: [--target-version, "4.2"]
 
-- repo: https://github.com/psf/black
-  rev: 23.7.0
-  hooks:
-    - id: black
-      # It is recommended to specify the latest version of Python
-      # supported by your project here, or alternatively use
-      # pre-commit's default_language_version, see
-      # https://pre-commit.com/#top_level-default_language_version
-      language_version: python3.10
-      exclude: |
-        (?x)^(
-            readthedocs/rtd_tests/files/conf.py|
-            .*/migrations/.*
-        )$
-
-# NOTE: run `isort` after `black` to keep the format of isort finally
-- repo: https://github.com/pycqa/isort
-  rev: 5.12.0
-  hooks:
-    - id: isort
-      name: isort (python)
-      exclude: |
-        (?x)^(
-            .*/migrations/.*
-        )
-
 - repo: https://github.com/asottile/blacken-docs
   rev: 1.16.0
   hooks:
     - id: blacken-docs
       additional_dependencies: [black==23.7.0]
 
-- repo: https://github.com/PyCQA/prospector
-  rev: 1.10.2
-  hooks:
-    - id: prospector
-      args:
-        - --profile=prospector.yml
-        - --die-on-tool-error
-      exclude: |
-        (?x)^(
-            docs/.*|
-            common/.*|
-            .*/settings/.*|
-            .*/rtd_tests/.*|
-            .*/tests/.*|
-            .*/migrations/.*
-        )$
-      # This PYTHONPATH environment variable is required for readthedocs-corporate when running `tox -e pre-commit`
-      entry: env PYTHONPATH=readthedocs:../readthedocs.org:./ DJANGO_SETTINGS_MODULE=readthedocs.settings.test prospector
-      # We have to install the dependencies from an URL since we don't have access to this file easily from pre-commit
-      additional_dependencies:
-        - -r
-        - https://raw.githubusercontent.com/readthedocs/readthedocs.org/main/requirements/testing.txt
-
 - repo: https://github.com/aspiredu/django-safemigrate
   rev: "4.2"
   hooks:
     - id: check
+
+- repo: https://github.com/astral-sh/ruff-pre-commit
+  # Ruff version.
+  rev: v0.9.9
+  hooks:
+    # Run the linter.
+    - id: ruff
+      args: [ --fix ]
+    # Run the formatter.
+    - id: ruff-format