Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Symfony 7.0 support #720

Closed
thenerka1 opened this issue Dec 1, 2023 · 1 comment · Fixed by #721
Closed

Symfony 7.0 support #720

thenerka1 opened this issue Dec 1, 2023 · 1 comment · Fixed by #721

Comments

@thenerka1
Copy link

No description provided.

morawskim added a commit to morawskim/RabbitMqBundle that referenced this issue Dec 17, 2023
@morawskim
php-amqplib#720 Add support for Symfony 7
3c55411
morawskim added a commit to morawskim/RabbitMqBundle that referenced this issue Dec 17, 2023
@morawskim
php-amqplib#720 Add support for Symfony 7
c92b9ce
@morawskim
Copy link
Contributor

I have the same problem.
Im my fork (diff is here) I added support for Symfony7, but my changes are not backward-compatible.

The base class for command requires ContainerInterface from Symfony namespace, but ServiceLocator implements ContainerInterface from Psr namespace.
Thats why I have not yet create PR. I am looking for solution.
Also I need to test other commands.

@morawskim morawskim mentioned this issue Dec 18, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for Symfony 7 morawskim/RabbitMqBundle
2 participants
@morawskim @thenerka1