diff --git a/composer.json b/composer.json index a06e194c..e6201be9 100644 --- a/composer.json +++ b/composer.json @@ -15,14 +15,14 @@ "symfony/event-dispatcher": "^4.0" }, "require-dev": { - "phpunit/phpunit": "^7.0", + "phpunit/phpunit": "^7.2", "symfony/filesystem": "^4.0", "psr/log": "1.0", - "symplify/easy-coding-standard": "^3.0", - "slam/php-cs-fixer-extensions": "^1.10", - "nette/utils": "^2.4", + "symplify/easy-coding-standard": "^4.4", + "slam/php-cs-fixer-extensions": "^1.16", + "nette/utils": "^2.5", "phpstan/phpstan": "^0.9", - "symplify/changelog-linker": "^3.0" + "symplify/changelog-linker": "^4.4" }, "autoload": { "psr-4": { diff --git a/easy-coding-standard.neon b/easy-coding-standard.neon deleted file mode 100644 index 4ff92407..00000000 --- a/easy-coding-standard.neon +++ /dev/null @@ -1,30 +0,0 @@ -includes: - - vendor/symplify/easy-coding-standard/config/psr2.neon - - vendor/symplify/easy-coding-standard/config/php71.neon - - vendor/symplify/easy-coding-standard/config/common.neon - - vendor/symplify/easy-coding-standard/config/clean-code.neon - - vendor/symplify/easy-coding-standard/config/symplify.neon - -checkers: - # Metrics - PHP_CodeSniffer\Standards\Generic\Sniffs\Files\LineLengthSniff: - absoluteLineLimit: 120 - PHP_CodeSniffer\Standards\Generic\Sniffs\Metrics\CyclomaticComplexitySniff: - absoluteComplexity: 6 - PHP_CodeSniffer\Standards\Generic\Sniffs\Metrics\NestingLevelSniff: - absoluteNestingLevel: 4 - - # Class should be Final or Abstract - - SlamCsFixer\FinalInternalClassFixer -parameters: - exclude_checkers: - - PHP_CodeSniffer\Standards\Generic\Sniffs\CodeAnalysis\AssignmentInConditionSniff - - Symplify\CodingStandard\Sniffs\DependencyInjection\NoClassInstantiationSniff - - skip: - SlevomatCodingStandard\Sniffs\TypeHints\TypeHintDeclarationSniff: - - tests/TestLogger.php - - SlamCsFixer\FinalInternalClassFixer: - # class with children - - src/Event/GitEvent.php diff --git a/ecs.yml b/ecs.yml new file mode 100644 index 00000000..1321cb4c --- /dev/null +++ b/ecs.yml @@ -0,0 +1,33 @@ +imports: + - { resource: 'vendor/symplify/easy-coding-standard/config/psr2.yml' } + - { resource: 'vendor/symplify/easy-coding-standard/config/php71.yml' } + - { resource: 'vendor/symplify/easy-coding-standard/config/common.yml' } + - { resource: 'vendor/symplify/easy-coding-standard/config/clean-code.yml' } + - { resource: 'vendor/symplify/easy-coding-standard/config/symplify.yml' } + +services: + # Class should be Final or Abstract + SlamCsFixer\FinalInternalClassFixer: ~ + +parameters: + exclude_checkers: + - 'PHP_CodeSniffer\Standards\Generic\Sniffs\CodeAnalysis\AssignmentInConditionSniff' + - 'Symplify\CodingStandard\Sniffs\DependencyInjection\NoClassInstantiationSniff' + + skip: + SlevomatCodingStandard\Sniffs\TypeHints\TypeHintDeclarationSniff: + - 'tests/TestLogger.php' + + Symplify\CodingStandard\Sniffs\CleanCode\ForbiddenStaticFunctionSniff: + # helper method + - 'src/GitWrapper.php' + # parent interface → nothing we can do about this + - 'src/Event/GitLoggerListener.php' + + SlamCsFixer\FinalInternalClassFixer: + # class with children + - 'src/Event/GitEvent.php' + + Symplify\CodingStandard\Sniffs\CleanCode\ForbiddenReferenceSniff: + # native PHP api → nothing we can do about this + - 'tests/StreamSuppressFilter.php' diff --git a/src/GitCommand.php b/src/GitCommand.php index 9c2758e6..ffe24236 100644 --- a/src/GitCommand.php +++ b/src/GitCommand.php @@ -183,11 +183,7 @@ public function getCommandLine() return $this->getCommand(); } - $command = array_merge( - [$this->getCommand()], - $this->buildOptions(), - $this->args - ); + $command = array_merge([$this->getCommand()], $this->buildOptions(), $this->args); return array_filter($command); } diff --git a/tests/GitWorkingCopyTest.php b/tests/GitWorkingCopyTest.php index 9f05b7a3..b6c016b2 100644 --- a/tests/GitWorkingCopyTest.php +++ b/tests/GitWorkingCopyTest.php @@ -18,7 +18,7 @@ final class GitWorkingCopyTest extends AbstractGitWrapperTestCase /** * Creates and initializes the local repository used for testing. */ - public function setUp(): void + protected function setUp(): void { parent::setUp(); @@ -403,9 +403,9 @@ public function testCommitWithAuthor(): void $git->add('commit.txt'); $git->commit([ - 'm' => 'Committed testing branch.', - 'a' => true, - 'author' => 'test ', + 'm' => 'Committed testing branch.', + 'a' => true, + 'author' => 'test ', ]); $output = $git->log();