metosin / metosin/compojure-api
Automatically push `context` parameters into endpoints when safe
Offen
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Clojure
- Sterne
- 1.1k
- Forks
- 146
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Presents more opportunities to infer static contexts.
(context "/foo" []
:path-params [id :- s/Str]
(PATCH "/" []
(ok id))
(GET "/" []
(ok id)))
=>
(context "/foo" []
(PATCH "/" []
:path-params [id :- s/Str]
(ok id))
(GET "/" []
:path-params [id :- s/Str]
(ok id)))
typed.clj.analyzer would be useful here as we both need to infer where locals occur and also do partial macroexpansion.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Lesen von typed.clj.analyzer, das im Issue zum Auffinden von Locals und zur Durchführung einer partiellen Makroexpansion genannt wird. Vergleiche das Kontextbeispiel mit den gewünschten expandierten Endpoint-Parametern; als erledigt gilt die sichere Verschiebung statischer Kontextparameter in jeden zutreffenden PATCH- und GET-Endpoint, ohne andere Fälle zu ändern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- clojure
- Bereich
- api, backend-api-design
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 30/100