Skip to content

feat: allow symfony 8#32

Merged
kbond merged 4 commits intozenstruck:1.xfrom
Chris53897:feature/symfony-8
Jan 8, 2026
Merged

feat: allow symfony 8#32
kbond merged 4 commits intozenstruck:1.xfrom
Chris53897:feature/symfony-8

Conversation

@Chris53897
Copy link
Contributor

@Chris53897 Chris53897 commented Nov 28, 2025

"zenstruck/browser" does not yet support symfony 8
i need to check if symfony 8 is already covered by ci

Blocked by:

@Chris53897 Chris53897 closed this Dec 5, 2025
@Chris53897 Chris53897 reopened this Dec 5, 2025
Copy link
Member

@kbond kbond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for getting started on this!

Co-authored-by: Kevin Bond <kevinbond@gmail.com>
Copy link

@94noni 94noni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

@Chris53897 Chris53897 closed this Jan 8, 2026
@Chris53897 Chris53897 reopened this Jan 8, 2026
"symfony/phpunit-bridge": "^6.0|^7.0",
"symfony/var-dumper": "^5.4|^6.0|^7.0",
"symfony/yaml": "^5.4|^6.0|^7.0",
"symfony/console": "^5.4|^6.0|^7.0|^8.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's I think a bug where the framework-bundle in 7.4+ always registers the CommandDataCollector (even if it isn't available). I need to look further but the quick solution is to just require console (pretty sure in 99% of apps, it's available).

@kbond kbond merged commit 002462f into zenstruck:1.x Jan 8, 2026
18 checks passed
@kbond
Copy link
Member

kbond commented Jan 8, 2026

Done and released!

@Chris53897
Copy link
Contributor Author

thanks

@Chris53897 Chris53897 deleted the feature/symfony-8 branch January 9, 2026 08:09
@94noni
Copy link

94noni commented Jan 9, 2026

thank you very much !

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.

4 participants