Skip to content

Conversation

@acrobat
Copy link

@acrobat acrobat commented Nov 4, 2025

Q A
Bug fix? no
New feature? yes
BC breaks? yes
Related tickets
License MIT

PR to prepare this package for newer symfony and doctrine versions and to unblock other dependencies down the road (sylius/resource-bundle for example)

  • Allow Symfony 8 and remove support for unmaintained Symfony 7 versions
  • Bump doctrine/data-fixtures to v2 to allow doctrine/persitence v4
    • With this bump doctrine/doctrine-bundle v3 can also be allowed which will unblock Symfony 8 when v3.1 is released

Dependencies with pending Symfony 8 support:

Dependency Info
doctrine/doctrine-bundle symfony 8 support available in 3.1.x-dev
doctrine/orm symfony 8 support available in 3.6.x-dev
nelmio/alice nelmio/alice#1284
theofidry/alice-data-fixtures Not compatible yet (no work started) theofidry/AliceDataFixtures#318

@lchrusciel lchrusciel closed this Nov 6, 2025
@lchrusciel lchrusciel reopened this Nov 6, 2025
lchrusciel pushed a commit that referenced this pull request Nov 6, 2025
Updated doctrine/data-fixtures from ^1.2 to ^2.1 to verify compatibility with PR #201.

Verification results:
- PHPStan analysis: PASSED (no errors)
- API compatibility: CONFIRMED (ORMPurger namespace and methods unchanged)
- Code compatibility: VERIFIED (all imports and usage patterns compatible)
- doctrine/persistence auto-upgraded to 4.1.1 (required by data-fixtures 2.x)

This change is part of the broader Symfony 8 and Doctrine dependencies update.
@lchrusciel
Copy link
Owner

Do you consider fixing build or should I take it over?

@acrobat acrobat force-pushed the bump-symfony-doctrine-deps branch 13 times, most recently from 71d65c6 to 9bba059 Compare November 7, 2025 08:11
@acrobat
Copy link
Author

acrobat commented Nov 7, 2025

Yeah sure, I will take a look at the tests! Most of them are passing now but some downstream dependencies still need a update for Symfony 8. I will first fix them and ping you when this PR is ready!

@acrobat acrobat marked this pull request as draft November 7, 2025 09:49
@acrobat acrobat force-pushed the bump-symfony-doctrine-deps branch 4 times, most recently from ba2d417 to 797d29d Compare November 7, 2025 10:48
@acrobat acrobat force-pushed the bump-symfony-doctrine-deps branch 4 times, most recently from c1f06a9 to 1a2ebc1 Compare November 7, 2025 10:58
@acrobat acrobat marked this pull request as ready for review November 7, 2025 11:11
@acrobat
Copy link
Author

acrobat commented Nov 7, 2025

@lchrusciel so far this PR is ready but to make the tests pass we're still waiting for upstream releases of dependencies. I've listed these dependencies in the PR description

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.

2 participants