Skip to content

Add support for Symfony 8#102

Open
nclavaud wants to merge 2 commits intoPostHog:masterfrom
nclavaud:feat/symfony_8
Open

Add support for Symfony 8#102
nclavaud wants to merge 2 commits intoPostHog:masterfrom
nclavaud:feat/symfony_8

Conversation

@nclavaud
Copy link
Copy Markdown

Hi there!

This PR fixes #101 by updating the symfony/clock dependency constraint to include version ^8.0, as the codebase is already compatible with this version.

Also, it removes the --verbose flag in the test script to ensure compatibility with PHPUnit v11.

Both changes are backward-compatible and improve the package's usability with newer versions of dependencies.

Thanks for reviewing!

The codebase only uses Clock methods compatible with Symfony 8.0.
Remove the --verbose flag that does not exists anymore in PHPUnit 11.
@phramz
Copy link
Copy Markdown

phramz commented Mar 10, 2026

Will this be merged any time soon?
IMO it is somehow urgent, since the missing SF8 support in the SDK is blocking all projects from upgrading as well forces new project to bootstrap on outdated Symfony versions.

@marandaneto marandaneto requested review from a team and cat-ph March 27, 2026 13:52
@marandaneto
Copy link
Copy Markdown
Member

marandaneto commented Mar 27, 2026

Commits must have verified signatures.

@nclavaud can you force push with signed commits?

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.

Support for Symfony 8.0

3 participants