Explicitly call ruff check

Now that ruff includes an autoformatter, `ruff <path>` is deprecated --
you must instead use `ruff check`.

Author: DavidCain

.github/workflows/ci.yml

@@ -8,7 +8,7 @@ jobs:
       fail-fast: false
       matrix:
         command:
-          - "ruff ."
+          - "ruff check ."
           - "ruff format --check ws"
           - "mypy ws"
           - "pylint --jobs=0 ws"

Makefile

@@ -53,15 +53,15 @@ check: lint test
 .PHONY: fix
 fix: install-python-dev
 	poetry run ruff format .
-	poetry run ruff --fix .
+	poetry run ruff check --fix .
 
 .PHONY: lint
 lint: lint-python typecheck-python lint-js
 
 .PHONY: lint-python
 lint-python: install-python-dev
 	poetry run ruff format --check .
-	poetry run ruff .
+	poetry run ruff check .
 	poetry run pylint --jobs 0 ws  # '0' tells pylint to auto-detect available processors
 
 .PHONY: typecheck-python

ws/tests/views/test_api_views.py

@@ -647,7 +647,6 @@ def _expect_members(self, *expected_members: MemberInfo) -> None:
         response = self.client.get("/stats/membership.json")
         resp_json = response.json()
         self.assertCountEqual(resp_json, {"members"})
-        # self.assertEqual(len(resp_json['members'], len(expected_members)))
         self.assertCountEqual(resp_json["members"], expected_members)
 
     @responses.activate

ws/utils/geardb.py

@@ -269,7 +269,7 @@ def update_affiliation(participant: models.Participant) -> requests.Response | N
 
 
 def _stats_only_all_active_members() -> list[MembershipInformation]:
-    """Yield emails and rental activity for all members with current dues."""
+    """Report emails & rental activity for all members with current dues."""
 
     return [
         MembershipInformation(