Skip to content

Conversation

@sstok
Copy link
Member

@sstok sstok commented Nov 28, 2025

Q A
Branch? main
Bug fix? no
New feature? yes
BC breaks? yes
Deprecations? no
Fixed tickets
License MIT

Support for Symfony 6 and 7 was removed, PHP 8.4 and Symfony 8.0 is now the minimum required version.

- Annotation PHPDoc was removed
- Change constructor arguments order for better DX
@sstok sstok force-pushed the refactor/symfony-8 branch 2 times, most recently from d0769c4 to 75af66e Compare November 28, 2025 20:48
@sstok
Copy link
Member Author

sstok commented Nov 28, 2025

Three times a charm...

@sstok sstok force-pushed the refactor/symfony-8 branch from 75af66e to f6c4d12 Compare November 28, 2025 20:50
@sstok sstok merged commit 9df9dac into rollerworks:main Nov 28, 2025
3 checks passed
@sstok sstok deleted the refactor/symfony-8 branch November 28, 2025 20:52
@sstok sstok changed the title Refactor/symfony 8 Symfony 8 support Nov 28, 2025
@VincentLanglet VincentLanglet mentioned this pull request Dec 17, 2025
sstok added a commit that referenced this pull request Dec 17, 2025
This PR was merged into the 3.0-dev branch.

Discussion
----------

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Fixed tickets | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

Hi @sstok, while I appreciate you worked on SF8 support, #71
I feel like the support for SF 7 (and 6 ?) was dropped to fast.

Currently, when using Symfony 7.4 we have the choice between
- Deprecations on 3.x branch because xmlLoader is used
- A 4.x branch which fixed this deprecation but doesn't support 7.4

All the (Breaking ?) changes you made for the 3.0.0 versions are still combatible with Symfony 7 ! (and symfony 6 too I think)
So you could keep at least 7.4 support.

Thanks for your consideration.

Commits
-------

6320927 Keep SF 7 support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant