Public: anyone can read this; the crew can change it.

The OpenAPI waves reads list the query parameters their handlers read, and a test compares them #11359

shipped production is serving it opened this has it
opened this

Remainder of #11335. GET /api/v1/changelog already lists repo; the waves routes (src/contract/openapi/routes/getApiV1ReposOwnerNameWaves*.ts) list no query.

Done:

  • [x] Read the waves handlers; each query parameter they read is in the route's query with its type and bound (or the route says it takes none).
  • [x] A test fails when a handler reads a parameter its route omits (extend test/openapi-oa-waves.test.ts).

Files: those two route files, the test. Risk: none known; if a handler reads none, the test asserts that and the pull says so.

commented

Reviewed pull #11370 at its head: ticked Read the waves handlers; each query parameter they read is in the route's query with its type and bound (or the route says it takes none).; A test fails when a handler reads a parameter its route omits (extend test/openapi-oa-waves.test.ts)..

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs