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

Update Eclipse-based formatters to latest versions #2179

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

jochenberger
Copy link
Contributor

@jochenberger jochenberger commented Jun 20, 2024

No description provided.

@jochenberger jochenberger force-pushed the update-groovy-eclipse branch 3 times, most recently from 0efbdd3 to afd3fb6 Compare June 20, 2024 09:43
@Goooler
Copy link
Member

Goooler commented Jun 20, 2024

Seems we can bump JDT and others as well, see #1939.

@jochenberger jochenberger force-pushed the update-groovy-eclipse branch from afd3fb6 to 3895af5 Compare June 20, 2024 11:28
@jochenberger
Copy link
Contributor Author

Seems we can bump JDT and others as well, see #1939.

Done

@jochenberger jochenberger changed the title Update Eclipse Groovy formatter to latest version Update Eclipse-based formatters to latest versions Jun 20, 2024
@@ -14,6 +14,9 @@ We adhere to the [keepachangelog](https://keepachangelog.com/en/1.0.0/) format (
* Bump default `ktlint` version to latest `1.2.1` -> `1.3.0`. ([#2165](https://github.com/diffplug/spotless/pull/2165))
* Bump default `ktfmt` version to latest `0.49` -> `0.51`. ([#2172](https://github.com/diffplug/spotless/pull/2172))
* Rename property `ktfmt` option `removeUnusedImport` -> `removeUnusedImports` to match `ktfmt`. ([#2172](https://github.com/diffplug/spotless/pull/2172))
* Bump default `eclipse` version to latest `4.29` -> `4.32`. ([#2179](https://github.com/diffplug/spotless/pull/2179))
* Bump default `greclipse` version to latest `4.49` -> `4.31`. ([#2179](https://github.com/diffplug/spotless/pull/2179))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We cannot update to 4.32 here yet, because, currently, the Groovy formatter version is tied to the Eclipse version and the matching plugin version has not been released yet (https://github.com/groovy/groovy-eclipse/milestone/25).

@Goooler Goooler requested a review from nedtwigg June 20, 2024 13:24
@nedtwigg nedtwigg merged commit 6fb9bca into diffplug:main Jun 20, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants