spec-first/connexion

Allow passing of a Specification object to AbstractApp.add_api

Open

#1 260 ouverte le 26 juin 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (738 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (4 346 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Inspectez la methode add api dans AbstractApp pour comprendre comment elle accepte actuellement spec (chemin de fichier ou dictionnaire). Modifiez la ensuite pour qu'elle accepte egalement un objet Specification, probablement en verifiant si l'entree est une instance de Specification et en l'utilisant directement.
Stack technique
python
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75