Skip to content

Allow Symfony 8.0#98

Merged
nicolas-grekas merged 1 commit intosymfony:mainfrom
alexandre-daubois:sf-8
Aug 5, 2025
Merged

Allow Symfony 8.0#98
nicolas-grekas merged 1 commit intosymfony:mainfrom
alexandre-daubois:sf-8

Conversation

@alexandre-daubois
Copy link
Member

@alexandre-daubois alexandre-daubois commented Aug 5, 2025

No description provided.

@nicolas-grekas
Copy link
Member

Let's split topics and do only the 8.0 allowance first.

@alexandre-daubois
Copy link
Member Author

Keeping old PHP versions seems incompatible. symfony/mercure/src/Hub.php uses CPP which breaks the tests

@nicolas-grekas
Copy link
Member

mercure says "php": ">=7.1.3",
CPP is a mistake

@alexandre-daubois
Copy link
Member Author

My bad, it's not CPP but the trailing comma I guess. Let me send a PR there as well

@alexandre-daubois
Copy link
Member Author

Here is the fix: symfony/mercure#125

@nicolas-grekas nicolas-grekas merged commit 79da14b into symfony:main Aug 5, 2025
20 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.

2 participants