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.

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

Update for Symfony 5 #65

Merged
merged 15 commits into from
Nov 13, 2020
Merged

Update for Symfony 5 #65

merged 15 commits into from
Nov 13, 2020

Conversation

BrandonFlude
Copy link
Contributor

Command must return an INT, often 0 - update Composer to accept 5.0.* versions of Symfony packages.

@Gemineye
Copy link

Gemineye commented Oct 1, 2020

Are there any plans to merge this PR?

@BrandonFlude
Copy link
Contributor Author

@Gemineye, awaiting response from the developer.

@Phobetor
Copy link
Owner

Phobetor commented Nov 13, 2020

@BrandonFlude thank you very much for your contribution!
Can you please remove the fork hint in the readme from the pull request?
It makes no sense to merge it.

@BrandonFlude
Copy link
Contributor Author

@Phobetor - sure can.

@Phobetor Phobetor merged commit 6930c41 into Phobetor:master Nov 13, 2020
@BrandonFlude
Copy link
Contributor Author

@Phobetor - any chance we can get a tag out for this so we can pull through composer with the latest version? Thanks!

@Gemineye
Copy link

@Phobetor - any chance we can get a tag out for this so we can pull through composer with the latest version? Thanks!

+1

@Phobetor
Copy link
Owner

Hi @BrandonFlude

Thank you very much for this pull request!

It is released in version 3.2.2.

(fyi @Gemineye )

@Gemineye
Copy link

@Phobetor thanks! Maybe you should remove the usage of kernel.root_dir. I had to add this paramter:

kernel.root_dir: '%kernel.project_dir%'

Mapsred pushed a commit to igraal/rabbitmq-supervisor-bundle that referenced this pull request Mar 18, 2021
add return values to commands
add symfony 5 instructions to readme
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.

3 participants