chore: migrate to ruff instead of pycodestyle and isort

Author: mariajgrimaldi

Makefile

@@ -58,9 +58,9 @@ upgrade: ## update the requirements/*.txt files with the latest packages satisfy
 quality: ## check coding style with pycodestyle and pylint
 	tox -e quality
 
-format: ## format code with black and isort
+format: ## format code with black and isort. Enable ruff to fix E (pycodestyle) and I (isort) issues
 	ruff format openedx_authz tests --line-length 120
-	ruff check --select I --fix --line-length 120
+	ruff check --select E,I --fix --line-length 120
 
 pii_check: ## check for PII annotations on all Django models
 	tox -e pii_check

requirements/dev.txt

@@ -224,8 +224,6 @@ pycasbin==2.2.0
     #   -r requirements/quality.txt
     #   casbin-django-orm-adapter
     #   redis-watcher
-pycodestyle==2.14.0
-    # via -r requirements/quality.txt
 pycparser==2.23
     # via
     #   -r requirements/quality.txt

requirements/quality.in

@@ -5,7 +5,5 @@
 -r test.txt               # Core and testing dependencies for this package
 
 edx-lint                  # edX pylint rules and plugins
-isort                     # to standardize order of imports
-pycodestyle               # PEP 8 compliance validation
 pydocstyle                # PEP 257 compliance validation
 ruff                      # Fast Python linter and formatter

requirements/quality.txt

@@ -122,9 +122,7 @@ iniconfig==2.1.0
     #   -r requirements/test.txt
     #   pytest
 isort==6.0.1
-    # via
-    #   -r requirements/quality.in
-    #   pylint
+    # via pylint
 jinja2==3.1.6
     # via
     #   -r requirements/test.txt
@@ -158,8 +156,6 @@ pycasbin==2.2.0
     #   -r requirements/test.txt
     #   casbin-django-orm-adapter
     #   redis-watcher
-pycodestyle==2.14.0
-    # via -r requirements/quality.in
 pycparser==2.23
     # via
     #   -r requirements/test.txt

ruff.toml

@@ -0,0 +1,17 @@
+# Ruff configuration for openedx-authz
+# See https://docs.astral.sh/ruff/configuration/
+
+line-length = 120
+
+[lint]
+# E: pycodestyle errors
+# I: isort import sorting
+select = ["E", "I"]
+
+exclude = [
+    ".git",
+    ".tox",
+    "migrations",
+    "__pycache__",
+    "*.pyc",
+]

tox.ini

@@ -5,9 +5,10 @@ envlist = py{311,312}-django{42,52}
 ; D001 = Line too long
 ignore = D001
 
-[pycodestyle]
-exclude = .git,.tox,migrations
-max-line-length = 120
+[ruff]
+# Configuration moved to ruff.toml
+# select = E (pycodestyle errors), I (isort)
+# max-line-length = 120
 
 [pydocstyle]
 ; D101 = Missing docstring in public class
@@ -75,9 +76,8 @@ commands =
     touch tests/__init__.py
     pylint openedx_authz tests test_utils manage.py setup.py
     rm tests/__init__.py
-    pycodestyle openedx_authz tests manage.py setup.py
+    ruff check openedx_authz tests test_utils manage.py setup.py
     pydocstyle openedx_authz tests manage.py setup.py
-    isort --check-only --diff tests test_utils openedx_authz manage.py setup.py
     make selfcheck
 
 [testenv:pii_check]