Skip to content

Commit 8529f16

Browse files
committed
fix ignored argument error in chapter 5
1 parent 38054fb commit 8529f16

5 files changed

Lines changed: 10 additions & 8 deletions

File tree

Chapter05/api-gateway/routers/aggregation.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@ async def aggregated_health(
7878
7979
Demonstrates parallel health checking of multiple services.
8080
"""
81+
_ = request
8182
return await fetch_aggregated_health(
8283
portal_client,
8384
reservation_client,
@@ -144,6 +145,7 @@ async def get_availability_summary(
144145
Returns slots grouped by day and time slot with counts,
145146
demonstrating data transformation in the gateway.
146147
"""
148+
_ = request
147149
try:
148150
return await fetch_availability_summary(
149151
reservation_client,

Chapter05/api-gateway/routers/proxy.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,7 @@ async def get_portal_home_page(
8282
name: Annotated[str, Body()] = "",
8383
) -> str:
8484
"""Get the portal home page with language selection."""
85+
_ = request
8586
try:
8687
return await fetch_portal_home(
8788
client, lang=lang, name=name
@@ -104,6 +105,7 @@ async def get_all_reservations(
104105
time_slot: Annotated[TimeSlot | None, Query()] = None,
105106
) -> list[dict]:
106107
"""Get all reservations with optional filtering."""
108+
_ = request
107109
try:
108110
return await get_all_available_slots(
109111
client, week_day=week_day, time_slot=time_slot
@@ -129,6 +131,7 @@ async def reserve_slot(
129131
],
130132
) -> dict:
131133
"""Reserve a slot."""
134+
_ = request
132135
try:
133136
return await make_reservation(
134137
client,

Chapter05/api-gateway/tests/conftest.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,9 @@ async def rate_limit_handler(
6868
@app.get("/test")
6969
@limiter.limit("2/minute")
7070
async def test_endpoint(
71-
request: Request, _response: Response
71+
request: Request, response: Response
7272
):
73-
del request
73+
_ = request, response
7474
return {"message": "ok"}
7575

7676
@app.get("/unlimited")

Chapter05/api-gateway/tests/test_app_integration.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ async def availability_summary(
9696
week_day: str | None = None,
9797
time_slot: str | None = None,
9898
):
99-
del request, response
99+
_ = request, response
100100
from routers.aggregation import fetch_availability_summary
101101

102102
return await fetch_availability_summary(
@@ -107,8 +107,8 @@ async def availability_summary(
107107

108108
@app.get("/aggregate/health")
109109
@limiter.limit("5/minute")
110-
async def health(request: Request, _response: Response):
111-
del request
110+
async def health(request: Request, response: Response):
111+
_ = request, response
112112
from routers.aggregation import fetch_aggregated_health
113113

114114
return await fetch_aggregated_health(

pyproject.toml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,3 @@ known-first-party = [
2424
"features", "shared",
2525
"routers", "services", "rate_limiter", "config", "container",
2626
]
27-
28-
[tool.ruff.lint.per-file-ignores]
29-
"Chapter05/api-gateway/routers/*.py" = ["ARG001"]

0 commit comments

Comments
 (0)