From 87a1553a6079ca4eaf38527fc43082ad11ffee63 Mon Sep 17 00:00:00 2001 From: Markus Staab Date: Mon, 30 Dec 2019 12:12:11 +0100 Subject: [PATCH 1/2] Added phpdoc --- .../Rector/Include_/AbsolutizeRequireAndIncludePathRector.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php b/packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php index d1015dda6916..a7712cd2f530 100644 --- a/packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php +++ b/packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php @@ -15,6 +15,9 @@ use Rector\RectorDefinition\RectorDefinition; /** + * Include/Require should be followed by absolute path. + * This Rector might introduce backwards incompatible code, when the include/require beeing changed depends on the current working directory. + * * @see https://github.com/symplify/CodingStandard#includerequire-should-be-followed-by-absolute-path * * @see \Rector\CodeQuality\Tests\Rector\Include_\AbsolutizeRequireAndIncludePathRector\AbsolutizeRequireAndIncludePathRectorTest From 1f5830f1140a0c3a98d3051eb032620fb7ed0019 Mon Sep 17 00:00:00 2001 From: Markus Staab Date: Mon, 30 Dec 2019 12:16:53 +0100 Subject: [PATCH 2/2] move phpdoc --- .../Include_/AbsolutizeRequireAndIncludePathRector.php | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php b/packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php index a7712cd2f530..8b181df96fb4 100644 --- a/packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php +++ b/packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php @@ -15,9 +15,6 @@ use Rector\RectorDefinition\RectorDefinition; /** - * Include/Require should be followed by absolute path. - * This Rector might introduce backwards incompatible code, when the include/require beeing changed depends on the current working directory. - * * @see https://github.com/symplify/CodingStandard#includerequire-should-be-followed-by-absolute-path * * @see \Rector\CodeQuality\Tests\Rector\Include_\AbsolutizeRequireAndIncludePathRector\AbsolutizeRequireAndIncludePathRectorTest @@ -26,7 +23,7 @@ final class AbsolutizeRequireAndIncludePathRector extends AbstractRector { public function getDefinition(): RectorDefinition { - return new RectorDefinition('include/require to absolute path', [ + return new RectorDefinition('include/require to absolute path. This Rector might introduce backwards incompatible code, when the include/require beeing changed depends on the current working directory.', [ new CodeSample( <<<'PHP' class SomeClass