Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added option to save the generated OpenAPI schema document on build. #11638

Merged
merged 1 commit into from
Aug 27, 2020
Merged

Added option to save the generated OpenAPI schema document on build. #11638

merged 1 commit into from
Aug 27, 2020

Conversation

phillip-kruger
Copy link
Member

@phillip-kruger phillip-kruger commented Aug 26, 2020

As discussed in Zulip a few times, this will add a new config option, quarkus.smallrye-openapi.store-schema-directory, that if
set, allow users to store the generated openapi.json and openapi.yaml to the configured directory

Signed-off-by:Phillip Kruger [email protected]

@phillip-kruger phillip-kruger requested a review from gsmet August 26, 2020 11:13
@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Aug 26, 2020
@gsmet gsmet merged commit 516e47a into quarkusio:master Aug 27, 2020
@gsmet gsmet added this to the 1.8.0 - master milestone Aug 27, 2020
@thomas-mc-work
Copy link
Contributor

How can this process be triggered?

@phillip-kruger
Copy link
Member Author

You can add a property quarkus.smallrye-openapi.store-schema-directory that points to the directory where you want to store the schema. Example:

quarkus.smallrye-openapi.store-schema-directory=target

@phillip-kruger phillip-kruger deleted the openapi-store-schema branch September 16, 2020 12:17
@thomas-mc-work
Copy link
Contributor

thomas-mc-work commented Sep 16, 2020

Three minutes … that was a fast reply!

I did that, followed by a mvn compile resp. mvn package. But that didn't yield the expected result.

@phillip-kruger
Copy link
Member Author

Are you using v 1.8.0 ?

@thomas-mc-work
Copy link
Contributor

Yes:

<properties>
	…
    <quarkus-plugin.version>1.8.0.Final</quarkus-plugin.version>
    <quarkus.platform.version>1.8.0.Final</quarkus.platform.version>
    …
</properties>

@thomas-mc-work
Copy link
Contributor

My fault! Somehow the quarkus-smallrye-openapi dependency got lost during a project refactoring.

Thank you very much, Phillip!

@phillip-kruger
Copy link
Member Author

Pleasure !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants