Apply suggestions from code review

Co-authored-by: Christian Schou Oxvig <[email protected]>

.github/workflows/CI_pull_request.yml

@@ -12,7 +12,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
         with:
-          python-version: "3.12"
+          python-version: "3.x"
       - uses: pre-commit/[email protected]
   run-full-test-suite:
     name: Run full test suite

cotainr/cli.py

@@ -51,7 +51,8 @@
 class CotainrSubcommand(ABC):
     """Abstract base class for `CotainrCLI` subcommands."""
 
-    @classmethod  # noqa: B027
+    @classmethod
+    @abstractmethod
     def add_arguments(cls, *, parser):
         """
         Add command line arguments to arguments `parser`.

ruff.toml

@@ -2,11 +2,11 @@ target-version = "py38"
 line-length = 88
 
 [lint]
-extend-select = [
-    "B",
-    "E",
-    "F",
-    "W",
+select = [
+    "B", # flake8-bugbear
+    "E", # pycodestyle errors
+    "F", # Pyflakes
+    "W", # pycodestyle warnings
 ]
 ignore = [
     "E501", # ignore line length; auto-formatting will take care of what it can.
@@ -20,15 +20,16 @@ exclude = [
     # The non-standard way to import py.test fixtures in
     # tests means we will need to ignore "imported but not used"
     # as well as argument shadowing tests in test modules.
-    # It would be better to use standard `conftest` style fixture
-    # discovery.
     "F401",
     "F811",
 ]
 "cotainr/tests/**.py" = [
     "S101", # assertions are OK in tests
 ]
 
+[lint.pydocstyle]
+convention = "numpy"
+
 [format]
 exclude = [
     "examples/**/*.ipynb",