Add phpcbf to help fix minor errors automatically about wprig

[ataylorme]

Any objection for this being a Composer script rather than a gulp task? That will be a simpler solution as PHPCS is installed via Composer. We might be able to make a gulp task that calls the Composer script as well.

[mor10]

Tentatively yes, though I'd like to see someone do a test to figure out if the fixing is clean or it it can cause new issues.

[jdelia]

I am using phpcbf with vscode and it is working.

[ataylorme]

After thinking on this I think we should use gulp. If we were to use PHPCBF directly of via Composer then it will attempt to fix all PHP files, including vendor.

We could list out which files to include and exclude but that would be duplicating the same declarations in gulp and we should not repeat ourselves in this way.

However, I tried using gulp-phpcbf but could not get it working.

I also tried changing the bin in [gulp-phpcs] (https://www.npmjs.com/package/gulp-phpcs) to be phpcbf but that did not work.

It seems we will either need to fork gulp-phpcs project to work with phpcbf or figure out what is wring with gulp-phpcbf, which hasn't been updated in 4 years.

Either way that is too much work than I can commit to. If someone else wants to take a stab at this please do.

[ataylorme]

@felixarntz has convinced me that my opinion in November is better than my opinion in December. So, not that it is January I am switching back to being in favor of this being in Composer, not gulp

[mor10]

What needs to be done to close this issue?

[ataylorme]

What needs to be done to close this issue?

A task similar to the existing run-phpcs needs to be added to the scripts section of composer.json that will run PHP code beautifier.