swagger-api/swagger-codegen

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

Aperta

#3100 aperta il 9 giu 2016

Vedi su GitHubGuida contributor
 (1 commento) (1 reazione) (0 assegnatari)HTML (5474 fork)batch import
Enhancement: Generalhelp wanted

Metriche repository

Star
 (12.701 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga il supporto esistente per .swagger codegen ignore e confrontalo con la logica writeOptional. Identifica dove viene usato writeOptional e proponi modifiche per usare il file di ignore.
Tech stack
java
Dominio
build system
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJava
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70