Skip to content

Conversation

@jaapio
Copy link
Member

@jaapio jaapio commented Dec 22, 2025

fixes #361, #362

Typed tags are no longer creatable via the create method. The
parsing has become to complex to handle per tag. This logic has been
moved into factories.
@jaapio jaapio linked an issue Dec 23, 2025 that may be closed by this pull request
TagWithType need to be created via a factory now.
@jaapio jaapio marked this pull request as ready for review December 24, 2025 15:17
@jaapio jaapio merged commit 00f1bda into 5.x Dec 24, 2025
49 checks passed
@xabbuh
Copy link
Contributor

xabbuh commented Dec 28, 2025

@jaapio Is it intended to get these changes in 5.x? Asking because I wonder if in Symfony we should adapt for these changes as a bugfix on all branches supporting 5.x or if that’s for a new feature supporting 6.x.

@jaapio
Copy link
Member Author

jaapio commented Dec 28, 2025

Whoops, that was wrong. Sorry for that.

@jaapio jaapio deleted the prepare-6.x-build branch December 28, 2025 10:05
@jaapio
Copy link
Member Author

jaapio commented Dec 28, 2025

I removed all the commits from the 5.x branch. Sorry for this mistake

@xabbuh
Copy link
Contributor

xabbuh commented Dec 28, 2025

Thank you! I’ll keep you posted if I discover any issues while allowing 6.x in the upcoming Symfony 8.1 branch.

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.

[DEPRECATION] Param tags without name [DEPRECATION] Usage of static factory methods

3 participants