swagger-api/swagger-codegen

[All] Use .swagger-codegen-ignore instead of writeOptional

Ouverte

#3 100 ouverte le 9 juin 2016

Voir sur GitHubGuide contributeur
 (1 commentaire) (1 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

I think .swagger-codegen-ignore is better since it can be configured by the end user whereas writeOptional files are hard-coded in the codegen. For instance, when generating the petstore samples, you want to overwrite the pom.xml but it doesn't occur unless you rm the file first. It would be better to pre-configure in the templates the .swagger-codegen-ignore to ignore pom.xml, build.gradle, etc... The only writeOptional file would then be .swagger-codegen-ignore

Guide contributeur

Direction de recherche
Examinez la prise en charge existante de .swagger codegen ignore et comparez la avec la logique writeOptional. Identifiez où writeOptional est utilisé et proposez des modifications pour utiliser le fichier d'ignore à la place.
Stack technique
java
Domaine
build system
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJava
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70