-
Notifications
You must be signed in to change notification settings - Fork 460
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
XML Extension #293
XML Extension #293
Conversation
Integrated WTP XML formatter. Fixed WTP formatter step naming (WtpEclipse -> EclipseWtp). Naming convention for all Eclipse formatter is Eclipse<Kind>...
The Eclipse formatter is described in a dedicated sub-section. The configFile example follows directly after the describing paragraph.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks great except the default includes, but in the end I think they're sensible too. Having those defaults will probably help a few users get started, and won't hurt pro users anyway, so you won me over on them :)
@lutovich I would appreciate if you have a quick look on the Maven integration. |
|
||
} | ||
|
||
/** If the user hasn't specified the files yet, we'll assume he/she means all of the C/C++ files. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
XML files
?
Prefer to remove the description. Code is simple enough.
plugin-maven/src/main/java/com/diffplug/spotless/maven/xml/Eclipse.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good!
This is awesome!!! |
@JLLeitschuh Just found some mistakes in the README.md. But before release I also want a problem with the Provisioner in the |
# Conflicts: # plugin-gradle/CHANGES.md # plugin-gradle/README.md # plugin-gradle/src/main/java/com/diffplug/gradle/spotless/SpotlessExtension.java # plugin-maven/CHANGES.md # plugin-maven/src/main/java/com/diffplug/spotless/maven/AbstractSpotlessMojo.java # plugin-maven/src/test/java/com/diffplug/spotless/maven/MavenIntegrationTest.java
Provided XML extension for Gradle and Maven.
Integrated Eclipse-WTP XML formatter step.
This PR implements #140 .
I took the liberty already to apply the formatter on Spotless itself. Hence the huge amount of changes.
Please follow the single commits for a better overview of the subjects addressed by the changes.