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

Provision of Spotless Eclipse CDT formatter plugin. #231

Merged
merged 1 commit into from
Apr 5, 2018
Merged

Conversation

fvgh
Copy link
Member

@fvgh fvgh commented Apr 4, 2018

No description provided.

@fvgh fvgh mentioned this pull request Apr 4, 2018
@nedtwigg
Copy link
Member

nedtwigg commented Apr 5, 2018

Fantastic! Like #230, I'm happy to merge now, and then work on UI and docs in a new PR if you prefer.

@fvgh fvgh merged commit 15216f5 into master Apr 5, 2018
@fvgh
Copy link
Member Author

fvgh commented Apr 5, 2018

@nedtwigg Please don't provide a release now. I will make some updates, integrating the eclipse-mock (I prefer the name eclipse-base) as discussed for #230 .

@fvgh fvgh deleted the ext_eclipse_cdt branch April 28, 2018 11:24
@nedtwigg
Copy link
Member

nedtwigg commented Aug 8, 2018

cdt and wtp are both published to mavencentral. Might take a few hours before they're available from mavencentral, but they're definitely available from jcenter right now. Just FYI, had to make a small change to the cdt pom to get mavencentral to accept it: f6bca4d

@nedtwigg
Copy link
Member

nedtwigg commented Aug 8, 2018

Btw, I played around with spotless-eclipse-base in a little test project the other day - great API, works really well. We should publicize this more with a blog post / tweet. If you have a blog we can publish there and tweet it, else I'm happy to write a post and publish from mine. I'm slammed at the moment, but hopefully can get to this in a month or so.

@fvgh
Copy link
Member Author

fvgh commented Aug 8, 2018

I am always happy if my work is useful for other people.Feel free to promote it.

Sorry for the missing Maven compliant fix for CDT. I did it for the others, but somehow forgot to align the CDT build properties.

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