formatter-maven-plugin icon indicating copy to clipboard operation
formatter-maven-plugin copied to clipboard

Enhanchement : Add a cleanup goal to launch Clean Up with a specific EclipseCleanUpRules in addition of format

Open GoogleCodeExporter opened this issue 9 years ago • 7 comments

Enhanchement : Add a cleanup goal to launch Clean Up with a specific 
EclipseCleanUpRules in addition of format

Tell me if it is not clear.

Original issue reported on code.google.com by [email protected] on 1 Jan 2014 at 12:56

GoogleCodeExporter avatar Mar 14 '15 02:03 GoogleCodeExporter

Would this address some of the trailing whitespace problems described in #23 ?

ctubbsii avatar May 13 '15 18:05 ctubbsii

Yes cleanup action would fix trailing space. Same applies on eclipse mars

--- Original Message ---

From: "Christopher Tubbs" [email protected] Sent: May 13, 2015 2:24 PM To: "revelc/formatter-maven-plugin" [email protected] Subject: Re: [formatter-maven-plugin] Enhanchement : Add a cleanup goal to launch Clean Up with a specific EclipseCleanUpRules in addition of format (#32)

Would this address some of the trailing whitespace problems described in #23 ?


Reply to this email directly or view it on GitHub: https://github.com/revelc/formatter-maven-plugin/issues/32#issuecomment-101767568

hazendaz avatar May 13 '15 23:05 hazendaz

I'm interested in this as well. Any clues where to start in eclipse code?

chonton-elementum avatar Apr 04 '16 19:04 chonton-elementum

Not too familiar with the Eclipse codebase, but it seems kinda scattered. Might be best to check in with the Eclipse community to see where the best place to start would be. Hopefully, the code for cleanup is modular enough to be reused here, but it might be a long shot.

ctubbsii avatar Apr 05 '16 17:04 ctubbsii

Clean Ups have not been externalised like the formatter. Best you could do, is execute Eclipse in Headless mode, e.g. using TEA.

ambition-consulting avatar Jul 11 '18 13:07 ambition-consulting

Damn this would be so handy

delanym avatar Jun 06 '22 10:06 delanym

FWIW, the whitespace removal is no longer necessary from this, since that feature was added natively without the use of Clean Up actions.

ctubbsii avatar Jun 06 '22 20:06 ctubbsii

Closing this as original need was addressed and if intent was a full cleanup, I think that is far beyond scope of the formatter plugin as it is not formatting at all.

hazendaz avatar Mar 17 '24 05:03 hazendaz