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

Support checking Java formatting #711

Merged
merged 2 commits into from
May 16, 2024

Conversation

armanbilge
Copy link
Member

The sbt-java-formatter plugin exposes a javafmtCheckAll command.

https://github.com/sbt/sbt-java-formatter

@mzuehlke
Copy link
Collaborator

Don't we need to add the plugin?

@mzuehlke
Copy link
Collaborator

And what about the commands? Similar to scalafmt?

@armanbilge
Copy link
Member Author

Don't we need to add the plugin?

The sbt-typelevel-ci plugin doesn't actually add the dependencies (same for scalafmt/scalafix/MiMa/etc.). Unfortunately we just need a centralized place to organize all the various commands so they are easy to toggle for users that do want to use those plugins.

And what about the commands? Similar to scalafmt?

Sorry, I didn't understand this question.

@mzuehlke
Copy link
Collaborator

Sorry, I didn't understand this question.

I wanted to refer to these aliases:

https://github.com/typelevel/sbt-typelevel/blob/main/core/src/main/scala/org/typelevel/sbt/TypelevelPlugin.scala#L84-L95

@armanbilge
Copy link
Member Author

Oh, right! We should also add it there. Thanks, nice catch.

@mergify mergify bot added the core label May 16, 2024
@armanbilge armanbilge merged commit bdc0e9a into typelevel:main May 16, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants