Skip to content

deps: allow api-platform/core ^5.0 - #1

Open
mwansinck wants to merge 1 commit into
masterfrom
allow-api-platform-5-1x
Open

mwansinck wants to merge 1 commit into
masterfrom
allow-api-platform-5-1x

Conversation

@mwansinck

Copy link
Copy Markdown

Wat

Verruimt de constraint op api-platform/core van ^3.0 || ^4.0 naar ^3.0 || ^4.0 || ^5.0.

Waarom

Routy gebruikt deze fork als dev-master as 1.4 en wil naar API Platform 5. De huidige constraint blokkeert dat; in Routy staat daarom tijdelijk de inline alias 5.0.3 as 4.4.99. Na deze merge kan dat gewoon ^5.0 worden.

Risico

Alleen composer.json wijzigt. De bundle verwijst in src/ en tests/ nergens naar een API Platform-class.

Getest

  • composer update --with api-platform/core:^5.0 lost op naar v5.0.3
  • PHPUnit: 40 tests, 65 assertions, groen
  • Routy zelf draait groen op 5.0.3 (PHPUnit 292 tests, PHPStan zonder fouten), met deze bundle ongewijzigd in gebruik

Relatie met upstream

Voor de 2.x-lijn van upstream staat Locastic#89 open (branch api-platform-5). Deze PR is de tegenhanger voor de 1.x-lijn op onze master, die de breaking changes van 2.0 niet bevat.

🤖 Generated with Claude Code

The bundle does not reference any API Platform class, so only the
constraint needs widening. The test suite passes against
api-platform/core 5.0.3.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant