Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Eclipse updates for 4 12#423

Merged
nedtwigg merged 10 commits into
masterfrom
eclipse-updates-for-4-12
Jul 26, 2019
Merged

Eclipse updates for 4 12#423
nedtwigg merged 10 commits into
masterfrom
eclipse-updates-for-4-12

Conversation

@fvgh

@fvgh fvgh commented Jul 24, 2019

Copy link
Copy Markdown
Member

Provided updates for Eclipse based formatters as they were released for Eclipse 4.12 / 2019-06
Following updates are provided:

@fvgh

fvgh commented Jul 24, 2019

Copy link
Copy Markdown
Member Author

@nedtwigg If you don't mind, I do this all in one PR. I would appreciate if you could use this branch to provide new versions for:

  • spotless-eclipse-cdt
  • spotless-eclipse-groovy
  • spotless-eclipse-wtp

I don't think that we need a new spotless-eclipse-jdt version. The behaviour has changed (actually some versions ago). Have a look at my comments on the UTs if you are interested.
Anyhow, don't see a need to change the code. Would propose to wait with a new versions when I (finally) finished my work on #292.

In the change logs I left a few TBDs. Will fill the blanks as soon as we have the new versions.

@fvgh

fvgh commented Jul 24, 2019

Copy link
Copy Markdown
Member Author

This PR will provide fix for #422 .

@nedtwigg

Copy link
Copy Markdown
Member

Looks great! In progress...

@nedtwigg

Copy link
Copy Markdown
Member

All three have been published to jcenter and mavencentral.

@fvgh

fvgh commented Jul 25, 2019

Copy link
Copy Markdown
Member Author

@nedtwigg Tested the new defaults with some projects, including JUnit. Since I used old code versions, nothing changed except for Groovy (see also changes in 1c44cda ).

FYI: I came across a known WTP JSON bug. Since it is already part of 3.8, I see no reason not to upgrade the default.

@fvgh fvgh requested a review from nedtwigg July 25, 2019 22:10
@nedtwigg nedtwigg merged commit 882296d into master Jul 26, 2019
@nedtwigg nedtwigg deleted the eclipse-updates-for-4-12 branch July 26, 2019 15:30
@nedtwigg

Copy link
Copy Markdown
Member

Released in x.24.0

@k-brooks k-brooks mentioned this pull request Oct 31, 2019
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.

2 participants