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 format command to CLI for source formatting #31281

Closed
wants to merge 2 commits into from

Conversation

gastaldi
Copy link
Contributor

- Add formatting feature to the Quarkus CLI for sources using the same formatting rules used in the Quarkus Core project
@quarkus-bot quarkus-bot bot added area/cli Related to quarkus cli (not maven/gradle/etc.) area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins labels Feb 18, 2023
@maxandersen
Copy link
Member

This seems like a prime candidate to have as a quarkus-fmt@ plugin similar to quarkus-kill@ ?

@gastaldi
Copy link
Contributor Author

@maxandersen Oh you mean after #31198 is merged?

@maxandersen
Copy link
Member

Well you can make it now. Until quarkus has plug-in you run it using jbang fmt@quarkusio

@gastaldi
Copy link
Contributor Author

Added to jbang catalog. Closing this for now

@gastaldi gastaldi closed this Feb 22, 2023
@quarkus-bot quarkus-bot bot added the triage/invalid This doesn't seem right label Feb 22, 2023
@gastaldi gastaldi deleted the format_sources branch February 22, 2023 23:37
@geoand
Copy link
Contributor

geoand commented Apr 12, 2023

Quarkus CLI does have pluggins now, does it not? Should we revive this?

@gastaldi
Copy link
Contributor Author

That would be nice

@geoand
Copy link
Contributor

geoand commented Apr 12, 2023

Actually, jbang alias from the Quarkus catalog are already supported as CLI plugins, so there is nothing to be done here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cli Related to quarkus cli (not maven/gradle/etc.) area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins triage/invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Introduce quarkus format (or quarkus fmt)
3 participants