[Renaming][PostRector] Handle skip path after defined at RenameClassRector

[samsonasik]

Given the following config in tests:

use Rector\Config\RectorConfig;
use Rector\Renaming\Rector\Name\RenameClassRector;
use Rector\Tests\Renaming\Rector\Name\RenameClassRector\Source\NewClass;
use Rector\Tests\Renaming\Rector\Name\RenameClassRector\Source\OldClass;

return static function (RectorConfig $rectorConfig): void {
    $rectorConfig
        ->ruleWithConfiguration(RenameClassRector::class, [
            OldClass::class => NewClass::class,
        ]);

    // skip the path after defined
    $rectorConfig->skip([
        RenameClassRector::class => [sys_get_temp_dir() . '/rector/tests_fixture_'],
    ]);
};

It not properly skipped, as there are 2 services:

• RenameClassRector
• ClassRenamingPostRector

The ClassRenamingPostRector apply the change already before it actually skipped because that read the configs:

rector-src/packages/PostRector/Rector/ClassRenamingPostRector.php

Lines 27 to 35 in 3c168f0

	public function enterNode(Node $node): ?Node
	{
	$oldToNewClasses = $this->renamedClassesDataCollector->getOldToNewClasses();
	if ($oldToNewClasses === []) {
	return $node;
	}
	return $this->classRenamer->renameNode($node, $oldToNewClasses);
	}

This PR try to fix it with add new interface: PostRectorDependencyInterface to define the dependencies of the PostRector when needed.

Fixes rectorphp/rector#7417

[samsonasik]

Fixed 🎉 /cc @grandmaster44

[samsonasik]

All checks have passed 🎉 @TomasVotruba it is ready for review.

[samsonasik]

Rebased.

[samsonasik]

All checks have passed 🎉 @TomasVotruba I think it is ready.

[TomasVotruba]

Thank you 👏