@TrystanLea This can land. Just be aware in future, patches really shouldn't be merged unless the tests pass - otherwise there's no point in having them :)

We should slowly go through these rules, and remove the exclude entries, fixing up the code.

Thanks @reedy I've added a few inline comments.

Are you just running this command but on a small number of files for the above changes?

phpcbf -p .

I'm just running 'composer fix' which runs on everything, excluding vendor and Lib.

The indenting issues is probably from files not having consistent (tabs or spaces) indenting, not to consistent levels. And without a specific config setting that, it may just be getting confused as part of doing other changes...

Seeing as it's not doing weird things on most of the files in the repo, we can probably just fix that by hand one time and should be good going forward.

Like I say, if we can decide whether we actually want tabs or spaces, getting that fixed should be easier.

Before we go any further with this, lets get json_i18b merged in first, seeing lots of merge conflicts