diff --git a/src/Task/PhpCsFixer.php b/src/Task/PhpCsFixer.php index dd9c1c0a..a223f866 100644 --- a/src/Task/PhpCsFixer.php +++ b/src/Task/PhpCsFixer.php @@ -71,11 +71,12 @@ public function run(ContextInterface $context): TaskResultInterface $this->formatter->resetCounter(); $arguments = $this->processBuilder->createArgumentsForCommand('php-cs-fixer'); - $arguments->add('--format=json'); + $arguments->add('--format=txt'); $arguments->add('--dry-run'); $arguments->addOptionalBooleanArgument('--allow-risky=%s', $config['allow_risky'], 'yes', 'no'); $arguments->addOptionalArgument('--cache-file=%s', $config['cache_file']); $arguments->addOptionalArgument('--config=%s', $config['config']); + $arguments->addOptionalArgument('--ansi', true); if ($rules = $config['rules']) { $arguments->add(sprintf( diff --git a/test/Unit/Task/PhpCsFixerTest.php b/test/Unit/Task/PhpCsFixerTest.php index bcf86cff..209a699e 100644 --- a/test/Unit/Task/PhpCsFixerTest.php +++ b/test/Unit/Task/PhpCsFixerTest.php @@ -115,8 +115,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--verbose', 'fix', ] @@ -128,9 +129,10 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', '--allow-risky=yes', + '--ansi', '--verbose', 'fix', ] @@ -142,9 +144,10 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', '--cache-file=cachefile', + '--ansi', '--verbose', 'fix', ] @@ -156,9 +159,10 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', '--config=config.php', + '--ansi', '--verbose', 'fix', ] @@ -170,8 +174,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--rules=foo,bar', '--verbose', 'fix', @@ -188,8 +193,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--rules='.json_encode($rules), '--verbose', 'fix', @@ -202,8 +208,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--using-cache=yes', '--verbose', 'fix', @@ -216,8 +223,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', 'fix', ] ]; @@ -228,8 +236,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--verbose', '--diff', 'fix', @@ -242,8 +251,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--verbose', 'fix', ] @@ -255,8 +265,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--verbose', 'fix', 'hello.php', @@ -270,8 +281,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(GitPreCommitContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--verbose', 'fix', 'hello.php', @@ -285,8 +297,9 @@ public function provideExternalTaskRuns(): iterable $this->mockContext(GitPreCommitContext::class, ['hello.php', 'hello2.php']), 'php-cs-fixer', [ - '--format=json', + '--format=txt', '--dry-run', + '--ansi', '--path-mode=intersection', '--verbose', 'fix',