spec-first/connexion

Allow passing of a Specification object to AbstractApp.add_api

Open

#1260 aperta il 26 giu 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (738 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (4346 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Description

Right now you can pass either a file path to a JSON swagger spec, or a parsed dictionary of one.

Expected behaviour

It would be nice if you could pass in a Specification object, since they can be quite expensive to create even from a dictionary - as much as 10s for ours! Enabling this could speed up startup times for large apps, as well as aid in testing where apps are created over and over.

Actual behaviour

It doesn't do allow this :) People instead monkeypatch (eg. https://github.com/zalando/connexion/issues/306#issuecomment-584258868) to work around this which is never ideal.

Guida contributor

Direzione di ricerca
Ispeziona il metodo add api in AbstractApp per capire come attualmente accetta spec (percorso file o dizionario). Quindi modificalo per accettare anche un oggetto Specification, probabilmente controllando se l'input è un'istanza di Specification e usandolo direttamente.
Tech stack
python
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75