Ned Twigg
Ned Twigg
Valid concern, but caching plus incremental build makes it less important. If you want everything that googleJavaFormat does, then googleJavaFormat() is already there for you. If you want just one...
Great idea! Happy to help get a PR for this merged.
I'm all for it, but I see some potential hangups, e.g. `Add @Override and @Deprecated tags` The only way to implement this feature is if the formatter has already compiled...
VERY cool!!
> I don't think I can set the npm path then, can I? You can set the npm path ([docs](https://github.com/diffplug/spotless/tree/main/plugin-gradle#npm-detection), [code](https://github.com/diffplug/spotless/blob/75fdd3ece5857687a05d0259df66f4de4661dabc/plugin-gradle/src/main/java/com/diffplug/gradle/spotless/FormatExtension.java#L501-L512)). I *think* it has to be set at configuration-time,...
`doFirst` runs *after* `@TaskAction`, which is surprising. It's only "first" compared to `doLast` actions, unfortunately. But agreed, there would be more flexibility if you could set the exe at execution...
Happy to take a PR for this.
For substitutions that have constant values (e.g. Module and Project) you can do this already via `replaceRegex`. To inject something dynamic like the filename or author, you'll have to create...
### Dependencies > It doesn't have that many optional dependencies, right? Definitely hundreds, and possibly thousands :D Spotless supports many versions of many formatters. For example, when you say `eclipse('4.10')`...
``I deleted and edited comments above (including edits to every commenter in this thread), to remove debate about programming methodlogy``. https://gitter.im/diffplug/spotless is a good place to chat, I'd like to...