Skip to content

Add support for Symfony 8#102

Merged
marandaneto merged 3 commits intoPostHog:masterfrom
nclavaud:feat/symfony_8
Mar 30, 2026
Merged

Add support for Symfony 8#102
marandaneto merged 3 commits intoPostHog:masterfrom
nclavaud:feat/symfony_8

Conversation

@nclavaud
Copy link
Copy Markdown
Contributor

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!

@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?

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

@nclavaud can you force push with signed commits?

@marandaneto done, thanks!

@nclavaud
Copy link
Copy Markdown
Contributor Author

Not sure how to fix the failing check Call Feature Flags Project Workflow, though 🤔

@marandaneto
Copy link
Copy Markdown
Member

Not sure how to fix the failing check Call Feature Flags Project Workflow, though 🤔

not needed, this is internal and non blocking
i have to merge #112 first then i can merge yours, thanks for the PR

@marandaneto marandaneto merged commit a242e2a into PostHog:master Mar 30, 2026
9 of 10 checks passed
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