Ned Twigg

Results 639 comments of Ned Twigg

Just a formatter. We'll merge anything that someone is actually using, but I don't think Spotless provides much benefit to static analysis tools. [See this discussion](https://github.com/diffplug/spotless/issues/719#issuecomment-710270652) for details.

From @rougsig in issue above: > The main functionality that I want to get from usage spotless with detekt is git ratchet. That's a good usecase! As mentioned earlier in...

Related to #323, but easier. No plans to implement, happy to take a PR.

We should fix this, but in the meantime adding `-debug` should make it possible to see where the problem is.

Sounds like you've got a `**/*` target that should be a `**/*.blah` target. This will get fixed by #1097, but it might be awhile before that PR lands.

This would work for the Gradle plugin, but not the Maven one, and that's fine with me! One way to incorporate something like this would be to add documentation to...

Ahh, yes. `confirmVersionAndGetAbsolutePath` is called as part of up-to-date checking... I really hope the Gradle depenency model learns to play nice with PyPI/NPM/RubyGems/etc someday...

The root changelog has this at the top. https://github.com/diffplug/spotless/blob/bd2195ecf70b92b065611b6378e612c770af24a4/CHANGES.md#L3-L8 Happy to take a PR which makes this louder somehow, or that improves integration with dependabot.

Until this is fixed, the following is a workaround: - `git rm -r --cached .` - delete every file in the index - `git add .` - put every file...

That's a good idea. The place to do the check is here: https://github.com/diffplug/spotless/blob/f1aba173af53b87802b51d49f20b2a3fc1373d49/lib/src/main/java/com/diffplug/spotless/generic/PipeStepPair.java#L157 The tricky part is that the `off/on` tags actually build a regex, and we already have public...