GeezerVison

Thanks, that's super helpful! Would we say tabs are a requirement for JSON btw? I know they're offered for other means, but I can't think of any time I've seen them as JSON is usually space pretty-printed 🤔

Tabs are definitely not a requirement to merge, I just want to make it easy for some someone else to add them later if they want.

Looks like it's still not happy, I'm seeing:

* What went wrong:
Could not determine the dependencies of task ':spotlessApply'.
> Could not create task ':spotlessJsonApply'.
   > Could not create task ':spotlessJson'.
      > Cannot invoke method simple() on null object

And given it's pretty close to what is used in i.e. KotlinExtension I'm not sure where to go from here 😓

Hey @nedtwigg was wondering if you or any of the team could spot why this doesn't seem to be working?

At least right now, the tests are failing because the PR needs a spotlessApply. https://app.circleci.com/pipelines/github/diffplug/spotless/1060/workflows/c9ee7f59-a168-4836-8f02-88809ebc3730/jobs/5994

Because of that I can't see a test failure stacktrace in CI. Seems unlikely but possible that json is some kind of keyword or special operator, maybe try json2 just to test? If that is the problem it might be best for the example code to just be format 'json', com.diffplug.spotless.JsonExtension.class { (I got the package wrong here, but you get the idea).

This is a useful feature, but I throttle my time on Spotless. I won't pull this locally and dig around, and I don't trust stacktraces that I can't verify for myself in our CI setup. I can be more helpful if you can reproduce whatever you are seeing locally in the public CI.

Thanks Ned - I appreciate the help! I've added some extra test cases to show the various bits I've tried, which shows a bit more what I've been seeing.

Let me know if you'd like --stacktrace --info added for more logging detail.

Thanks Ned - good points for those documentation tweaks 👍

Published in plugin-gradle 5.14.0.

Hi, Is this available in plugin-maven as well?.

I don't see it there (yet) - I don't use Maven so didn't contribute it, but I can see about getting a PR raised at some point in the next few weeks, or if you'd like to pick it up, feel free :)

Any one know if this (along with #910) will be merged/released soon?

Hey talios, do you mean the Gradle or Maven plugin? Gradle been live for a while, the Maven and YAML changes won't be done at least by me any time soon sorry

@jamietanna Ahh I was meaning the maven plugin - if it's just a matter of wiring things up from library->mojo I could probably take a look at that ca814b4 commit and see what's up.