Skip to content

Allow Symfony v8 components#218

Merged
otsch merged 1 commit into
crwlrsoft:mainfrom
chr-hertel:patch-1
Apr 30, 2026
Merged

Allow Symfony v8 components#218
otsch merged 1 commit into
crwlrsoft:mainfrom
chr-hertel:patch-1

Conversation

@chr-hertel
Copy link
Copy Markdown
Contributor

Changes of Symfony 8

see https://github.com/symfony/symfony/blob/8.1/UPGRADE-8.0.md

* Component `symfony/css-selector` has zero breaking changes
* Component `symfony/dom-crawler` only has one irrelevant breaking change from code point of view, see symfony/symfony#61475 - not sure about behavior tho

see https://github.com/symfony/symfony/blob/8.1/UPGRADE-8.0.md
@otsch
Copy link
Copy Markdown
Member

otsch commented Apr 30, 2026

Hey @chr-hertel,
thanks a lot for the PR! Even though CI is failing on 8.1 for dependency conflict reasons with dev dependencies, I'll merge it. I'll find some solution for CI before tagging a release with this change.

@otsch otsch merged commit 14e7eba into crwlrsoft:main Apr 30, 2026
2 of 8 checks passed
@chr-hertel chr-hertel deleted the patch-1 branch April 30, 2026 11:31
@otsch
Copy link
Copy Markdown
Member

otsch commented May 2, 2026

Hey @chr-hertel,

thx again, I just released it with https://github.com/crwlrsoft/crawler/releases/tag/v3.5.7

Also the upgrade in the schema-org package with https://github.com/crwlrsoft/schema-org/releases/tag/v0.4.0 - I used the fact that it's still 0.x to just drop support for PHP 8.0 and 8.1 as well as for symfony/dom-crawler 6.x.

@chr-hertel
Copy link
Copy Markdown
Contributor Author

Awesome, thanks for the fast follow-up @otsch - really appreciated! 🙏

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