Skip to content

Narrowed rector/rector dependency below 2.6.2 - #51

Merged
konradoboza merged 1 commit into
5.0from
narrowed-rector-rector-below-2-6-2
Aug 12, 2026
Merged

Narrowed rector/rector dependency below 2.6.2#51
konradoboza merged 1 commit into
5.0from
narrowed-rector-rector-below-2-6-2

Conversation

@bnowak

@bnowak bnowak commented Aug 12, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue N/A

Description:

rector/rector 2.6.2 (released 2026-08-12) ships a rector/rector-symfony build that removed the deprecated version-based set constants SymfonySetList::SYMFONY_50SYMFONY_74 (rectorphp/rector-symfony#1019). IbexaRectorConfigFactory and the rector.php configs of downstream Ibexa packages still reference those constants, so any fresh composer install now fails with:

[ERROR] Undefined constant Rector\Symfony\Set\SymfonySetList::SYMFONY_60

This PR narrows the rector/rector requirement to ^2.0, <2.6.2 so that composer resolves 2.6.1 — the last working release — keeping this package and the CI jobs of all packages depending on it green.

Migrating IbexaRectorConfigFactory and per-package rector configs to the composer-based sets (->withComposerBased(symfony: true)) is a follow-up topic; this change only unblocks CI.

For QA:

Nothing to test manually — this is a dev-dependency version cap. Verified by composer update rector/rector resolving 2.6.2 => 2.6.1 and the unit test suite passing (68 tests, 79 assertions), including IbexaRectorConfigFactoryTest which exercises the removed constants.

Documentation:

No client-facing impact to document.

rector/rector 2.6.2 ships a rector-symfony build that removed the
deprecated version-based SymfonySetList constants (SYMFONY_50..SYMFONY_74),
which IbexaRectorConfigFactory and downstream package rector configs still
reference. Capping at <2.6.2 keeps packages and their CI jobs working until
the configs are migrated to composer-based sets.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@bnowak
bnowak marked this pull request as ready for review August 12, 2026 09:56

@Steveb-p Steveb-p left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked.

You can define multiple ranges. Ranges separated by a space ( ) or comma (,) will be treated as a logical AND. A double pipe (||) will be treated as a logical OR. AND has higher precedence than OR.

@konradoboza
konradoboza merged commit e80ca15 into 5.0 Aug 12, 2026
4 checks passed
@konradoboza
konradoboza deleted the narrowed-rector-rector-below-2-6-2 branch August 12, 2026 10:02
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.

4 participants