search

springdoc / springdoc-openapi-maven-plugin

Library for OpenAPI 3 with spring-boot
https://springdoc.org
Apache License 2.0
141 stars 37 forks source link

Adding support for yaml output #15

Closed anthropaedic closed 4 years ago

bnasslahsen commented 4 years ago

Thank you @anthropaedic for your contribution. For the moment, this enhancement will not be integrated as the existing configuration allows to achieve the PR goal.

patpatpat123 commented 4 years ago

@bnasslahsen But it is achieving it in a rather unfriendly manner. First of all, there are no clear indication of this magical -Dspringdoc.writer-with-default-pretty-printer=true Second, it is rather magical to have those http://localhost:8080/v3/api-docs.yaml

openapi.yaml to decide a yaml output. Would it be possible to have a more objective judgement on your statement "the existing configuration allows to achieve the PR goal". While it is true manipulating rather not descriptive properties can have the same output, (well, I an also feed the json to a python converter that will pretty print and convert to yaml on a separate job, it "does the job"). Having more descriptive, self explanatory, proper configuration directly on the maven plugin block without hidden magical -Dparameter would be an improvement worth "not just closing". Thank you