Skip to content

Update classes for Symfony 7.X.X - #15

Open
ColonelMoutarde wants to merge 4 commits into
BedrockStreaming:masterfrom
ColonelMoutarde:feat/upgrade-for-symfony-7
Open

Update classes for Symfony 7.X.X#15
ColonelMoutarde wants to merge 4 commits into
BedrockStreaming:masterfrom
ColonelMoutarde:feat/upgrade-for-symfony-7

Conversation

@ColonelMoutarde

@ColonelMoutarde ColonelMoutarde commented Mar 11, 2026

Copy link
Copy Markdown

remove deprecated fixed :

Method "Symfony\Component\DependencyInjection\Extension\ExtensionInterface::load()" might add "void" as a native return type declaration in the future. 
Do the same in implementation "M6Web\Bundle\MonologExtraBundle\DependencyInjection\M6WebMonologExtraExtension" now to avoid errors or add an explicit @return annotation to suppress this message.

@ColonelMoutarde
ColonelMoutarde requested a review from a team as a code owner March 11, 2026 13:25
… compatibility

Update classes for Symfony 7.X.X
remove deprecated :
Method "Symfony\Component\DependencyInjection\Extension\ExtensionInterface::load()" might add "void" as a native return type declaration in the future. Do the same in implementation "M6Web\Bundle\MonologExtraBundle\DependencyInjection\M6WebMonologExtraExtension" now to avoid errors or add an explicit @return annotation to suppress this message.
@ColonelMoutarde
ColonelMoutarde force-pushed the feat/upgrade-for-symfony-7 branch from 6f8c86b to aba8ffe Compare March 13, 2026 14:15

@shavounet shavounet left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This project would use a bit of PHPStan & Rector I suppose...
But anyway, LGTM

@ColonelMoutarde

Copy link
Copy Markdown
Author

This project would use a bit of PHPStan & Rector I suppose... But anyway, LGTM

Done

@ColonelMoutarde

Copy link
Copy Markdown
Author

Please, merge this PR

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.

5 participants