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

Add closeBootstrappedApp mojo param to QuarkusBootstrapMojo #24634

Merged
merged 1 commit into from Apr 5, 2022

Conversation

aloubyansky
Copy link
Member

#24026 apparently resulted in quarkus:generate-code invoked from the DevMojo to close the bootstrapped app because the Maven session listener isn't initialized from the invoker, making the DevMojo to bootstrap it again.
This PR introduces a parameter that allows DevMojo to instruct quarkus:generate-code to keep the bootstrapped app open.

@quarkus-bot quarkus-bot bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/maven labels Mar 30, 2022
@quarkus-bot

This comment has been minimized.

@quarkus-bot

This comment has been minimized.

@famod famod closed this Mar 31, 2022
@quarkus-bot quarkus-bot bot added triage/invalid This doesn't seem right and removed triage/backport-2.7? labels Mar 31, 2022
@famod famod reopened this Mar 31, 2022
@quarkus-bot quarkus-bot bot removed the triage/invalid This doesn't seem right label Mar 31, 2022
@famod
Copy link
Member

famod commented Mar 31, 2022

Sorry, wrong button.

@gsmet could it be that the moved test-extension is missing disable-unbind-executions?

@quarkus-bot

This comment has been minimized.

Copy link
Member

@famod famod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

At some point we should align existing property names.

@aloubyansky aloubyansky merged commit 7979f1b into quarkusio:main Apr 5, 2022
@quarkus-bot quarkus-bot bot added this to the 2.9 - main milestone Apr 5, 2022
@gsmet gsmet modified the milestones: 2.9 - main, 2.8.1.Final Apr 11, 2022
@gsmet gsmet modified the milestones: 2.8.1.Final, 2.7.6.Final May 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/maven
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants