Skip to content

[assert] make AddAssertArrayFromClassMethodDocblockRector rule configurable to accept beberlei assert package#7215

Merged
TomasVotruba merged 1 commit intomainfrom
tv-assert-config
Sep 4, 2025
Merged

[assert] make AddAssertArrayFromClassMethodDocblockRector rule configurable to accept beberlei assert package#7215
TomasVotruba merged 1 commit intomainfrom
tv-assert-config

Conversation

@TomasVotruba
Copy link
Member

@TomasVotruba TomasVotruba commented Sep 4, 2025

Based on feedback on Twitter, I've added another option to enable beberlei/assert package.

  1. to use it, configure the rule
use Rector\Assert\Enum\AssertClassName;
use Rector\Assert\Rector\ClassMethod\AddAssertArrayFromClassMethodDocblockRector;
use Rector\Config\RectorConfig;

return static function (RectorConfig $rectorConfig): void {
    $rectorConfig->ruleWithConfiguration(AddAssertArrayFromClassMethodDocblockRector::class, [
        AssertClassName::BEBERLEI,
    ]);
};
  1. to use webmozart/assert, use default or configure as well
use Rector\Assert\Rector\ClassMethod\AddAssertArrayFromClassMethodDocblockRector;
use Rector\Config\RectorConfig;

return static function (RectorConfig $rectorConfig): void {
    $rectorConfig->rule(AddAssertArrayFromClassMethodDocblockRector::class);

    // or 
    // $rectorConfig->ruleWithConfiguration(AddAssertArrayFromClassMethodDocblockRector::class, [
    //    AssertClassName::WEBMOZART,    
    // ]);
};

@TomasVotruba TomasVotruba force-pushed the tv-assert-config branch 3 times, most recently from d22312d to 85703bd Compare September 4, 2025 12:50
@TomasVotruba TomasVotruba changed the title make assert rule configurable [assert] make AddAssertArrayFromClassMethodDocblockRector rule configurable to accept beberlei assert package Sep 4, 2025
@TomasVotruba TomasVotruba merged commit edfd9f8 into main Sep 4, 2025
49 checks passed
@TomasVotruba TomasVotruba deleted the tv-assert-config branch September 4, 2025 13:01
arshidkv12 pushed a commit to arshidkv12/rector-src that referenced this pull request Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant