4.x move to Command class

[saeideng]

No description provided.

[lorenzo]

Why did you remove the whitespace shell?

[saeideng]

Why did you remove the whitespace shell?

because there are proper code linting tools

[saeideng]

I can revert it
it just says/shows for example

!!!contains leading whitespaces: /src/Controller/PagesController.php
!!!contains trailing whitespaces: /src/Controller/PagesController.php

[stickler-ci]

Could not review pull request. It may be too large, or contain no reviewable changes.

[lorenzo]

Yeah, I know the output. The idea is that it points you out to places where extra whitespace is dangerous.

[saeideng]

OK
https://travis-ci.org/cakephp/debug_kit/builds/444788571
I'll work on this PR, after opening a PR for making compatible debug_kit with cache's changes

[markstory]

@lorenzo I recommended it be removed as the whitespace issues can be caught by PHPCS now, and I think that it is a better tool for this kind of problem.

[saeideng]

let me know , should I revert the WhitespaceShell ? and convert it to Command

[markstory]

I think tools like phpcs are better suited for detecting trailing/leading whitespace and it isn't something that debugkit is doing better.

[saeideng]

so , this PR can merge