swagger-api/swagger-codegen

Review isDataTypeFile, isDataTypeBinary in all generators

Ouverte

#5 103 ouverte le 17 mars 2017

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)HTML (5 474 forks)batch import
Enhancement: Generalhelp wanted

Métriques du dépôt

Stars
 (12 701 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Description

Similar to https://github.com/swagger-api/swagger-codegen/issues/4764, we'll need to review other generators to make sure isDataTypeFile and isDataTypeBinary are properly overridden.

Swagger-codegen version

Latest master

Suggest a Fix

The fix should be similar to https://github.com/swagger-api/swagger-codegen/pull/5065

If anyone wants to help on this, please reply to let us know.

Guide contributeur

Direction de recherche
Consultez le PR #5065 pour le modèle de correction et vérifiez toutes les classes de générateur dans la base de code pour la surcharge correcte des méthodes isDataTypeFile et isDataTypeBinary.
Stack technique
java
Domaine
apibackendtooling
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaSwagger Codegen
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50