Add GET /api/v1/commitfests endpoint listing all commitfests (#114)

There's no way to discover which commitfests exist. `needs_ci` returns
only the two or three currently relevant ones, and
`/api/v1/commitfests/<cfid>/patches` requires a cfid.

This adds `GET /api/v1/commitfests` which returns every commitfest,
closed and draft ones included:

```json
{"commitfests": [{"id": 1, "name": "PG18-3", "status": "Closed", "draft": false,
                  "startdate": "2024-11-01", "enddate": "2024-11-30"}, ...]}
```

This data is all already available from `/commitfest_history` and
`/archive`, but those would need to be scraped, this makes the data
machine-readable.

Notes:

- Order by ID since start dates can tie and this gives us a stable
order.
- No pagination because it's a small dataset.
- I've exposed the 'draft' field through `CommitFest.to_json()`. This
does add the key to `needs_ci`. If we'd rather be strict about that
endpoint, I can build the dict inline in a new view instead.

Author: kjrocker

.github/workflows/ci.yaml

@@ -14,10 +14,13 @@ jobs:
 
       - name: Run ruff check
         uses: astral-sh/ruff-action@v2
+        with:
+          version: 0.15.20
 
       - name: Run ruff format --check
         uses: astral-sh/ruff-action@v2
         with:
+          version: 0.15.20
           args: "format --check"
 
       - name: Setup Biome

pgcommitfest/commitfest/apiv1.py

@@ -36,6 +36,11 @@ def api_response(payload, status=200, content_type="application/json"):
     return response
 
 
+def all_commitfests(request):
+    """Return all commitfests, including closed ones."""
+    return api_response({"commitfests": list(CommitFest.objects.order_by("id"))})
+
+
 def commitfestst_that_need_ci(request):
     cfs = CommitFest.relevant_commitfests()
 

pgcommitfest/commitfest/models.py

@@ -106,6 +106,7 @@ def to_json(self):
             "id": self.id,
             "name": self.name,
             "status": self.statusstring,
+            "draft": self.draft,
             "startdate": self.startdate.isoformat(),
             "enddate": self.enddate.isoformat(),
         }

pgcommitfest/commitfest/tests/test_apiv1.py

@@ -12,6 +12,70 @@
 pytestmark = pytest.mark.django_db
 
 
+def test_commitfests_endpoint(client, commitfests):
+    """Test the /api/v1/commitfests endpoint returns all commitfests."""
+    response = client.get("/api/v1/commitfests")
+
+    assert response.status_code == 200
+    assert response["Content-Type"] == "application/json"
+    assert response["Access-Control-Allow-Origin"] == "*"
+
+    data = json.loads(response.content)
+
+    expected = [
+        {
+            "id": commitfests["open"].id,
+            "name": "2025-01",
+            "status": "Open",
+            "draft": False,
+            "startdate": "2025-01-01",
+            "enddate": "2025-01-31",
+        },
+        {
+            "id": commitfests["in_progress"].id,
+            "name": "2024-11",
+            "status": "In Progress",
+            "draft": False,
+            "startdate": "2024-11-01",
+            "enddate": "2024-11-30",
+        },
+        {
+            "id": commitfests["recent_previous"].id,
+            "name": "2024-09",
+            "status": "Closed",
+            "draft": False,
+            "startdate": "2024-09-01",
+            "enddate": "2024-09-30",
+        },
+        {
+            "id": commitfests["old_previous"].id,
+            "name": "2024-07",
+            "status": "Closed",
+            "draft": False,
+            "startdate": "2024-07-01",
+            "enddate": "2024-07-31",
+        },
+        {
+            "id": commitfests["draft"].id,
+            "name": "2025-03-draft",
+            "status": "Open",
+            "draft": True,
+            "startdate": "2025-03-01",
+            "enddate": "2025-03-31",
+        },
+    ]
+
+    assert data == {"commitfests": sorted(expected, key=lambda cf: cf["id"])}
+
+
+def test_commitfests_endpoint_empty(client):
+    """Test the /api/v1/commitfests endpoint with no commitfests."""
+    response = client.get("/api/v1/commitfests")
+
+    assert response.status_code == 200
+    assert json.loads(response.content) == {"commitfests": []}
+
+
 def test_needs_ci_endpoint(client, commitfests):
     """Test the /api/v1/commitfests/needs_ci endpoint returns correct data."""
     response = client.get("/api/v1/commitfests/needs_ci")
@@ -30,20 +94,23 @@ def test_needs_ci_endpoint(client, commitfests):
                 "id": commitfests["open"].id,
                 "name": "2025-01",
                 "status": "Open",
+                "draft": False,
                 "startdate": "2025-01-01",
                 "enddate": "2025-01-31",
             },
             "in_progress": {
                 "id": commitfests["in_progress"].id,
                 "name": "2024-11",
                 "status": "In Progress",
+                "draft": False,
                 "startdate": "2024-11-01",
                 "enddate": "2024-11-30",
             },
             "draft": {
                 "id": commitfests["draft"].id,
                 "name": "2025-03-draft",
                 "status": "Open",
+                "draft": True,
                 "startdate": "2025-03-01",
                 "enddate": "2025-03-31",
             },

pgcommitfest/urls.py

@@ -16,6 +16,7 @@
 
 urlpatterns = [
     re_path(r"^$", views.home),
+    re_path(r"^api/v1/commitfests$", apiv1.all_commitfests),
     re_path(r"^api/v1/commitfests/needs_ci$", apiv1.commitfestst_that_need_ci),
     re_path(r"^api/v1/commitfests/(\d+)/patches$", apiv1.commitfest_patches),
     re_path(r"^api/v1/patches/(\d+)/threads$", apiv1.patch_threads),