-
Notifications
You must be signed in to change notification settings - Fork 573
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
Implement Spotless formatting #6100
Conversation
Release note changesNo release note changes were detected. If you made changes that should be |
Generated by 🚫 Danger |
Unit Test Results 954 files ±0 954 suites ±0 34m 44s ⏱️ -12s For more details on these failures, see this check. Results for commit 10e880d. ± Comparison against base commit 4db5dc8. ♻️ This comment has been updated with latest results. |
Size Report 1Affected Products
Test Logs |
Coverage Report 1Affected Products
Test Logs |
NO_RELEASE_CHANGELOG
This may require a lot of PRs to rebase/merge. As a resolution, outstanding PRs could add Spotless themselves like done in the first commit and run
spotlessApply
before the merge. Perhaps adding Spotless should be its own commit to main so outstanding branches can just pull main and cherry pick in?Updated root documentation, sessions and vertex both still link to ktfmt for their formatting, how do we want to update that?
Google Java Format was updated to latest.
ktfmt
was not. We are currently using a version from 2022,0.41
. Latest is0.51
, however, upgrading will require a reformat. Do we want to pre-emptively upgrade now and format with the rest of these formatting changes? Current latest release of Spotless does not have configuration for ktfmt trailing comma formatting, which was added recently. It's committed to the repo 3 weeks ago but they haven't had a release.