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

Allow extension to define CLI plugins with aliases #40580

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

iocanel
Copy link
Contributor

@iocanel iocanel commented May 12, 2024

Currently, when an extension defines a plugin, the name is inferred by the artifact id. In some cases that can be annoying and this pull request addresses that, by optionally allowing user to specify the alias they need.

@quarkus-bot quarkus-bot bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/platform Issues related to definition and interaction with Quarkus Platform labels May 12, 2024
@iocanel
Copy link
Contributor Author

iocanel commented May 12, 2024

cc @metacosm AFAIR, this is something that you mentioned that you'd like to have.

@quarkus-bot

This comment has been minimized.

@metacosm
Copy link
Contributor

Indeed, that would be useful. Thank you!

@iocanel iocanel force-pushed the cli-plug-ext-alias branch 3 times, most recently from 7bc5014 to d905697 Compare May 16, 2024 06:03
@quarkus-bot

This comment has been minimized.

@geoand
Copy link
Contributor

geoand commented May 18, 2024

@iocanel this needs a formatting fix

@quarkus-bot

This comment has been minimized.

@geoand
Copy link
Contributor

geoand commented May 29, 2024

That Windows failure looks suspicious

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/platform Issues related to definition and interaction with Quarkus Platform triage/flaky-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants