-
Notifications
You must be signed in to change notification settings - Fork 12
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
Symfony5 support #36
Symfony5 support #36
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
even if I'm surprised that we don't have anything more to update...
but since you said that it works fine only with these few changes 🤷♂
I think that we didn't try to use this bundle from a SF5 application yet... so I can only trust you, right now
@Oliboy50 What are the next steps? Is there someone else who should take a look? How far are we to create a new tag? Thanks a lot! |
@Adrians-G our CI seems to run tests against the symfony 4.4 dependencies would it be possible to make it run also tests against symfony 5+? |
Co-Authored-By: Oliver THEBAULT <Oliboy50@users.noreply.github.com>
Co-Authored-By: Oliver THEBAULT <Oliboy50@users.noreply.github.com>
Co-Authored-By: Oliver THEBAULT <Oliboy50@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it seems that we'll have to use "sensiolabs/security-checker": "^6.0"
in our require-dev
section to allow travis to use symfony/console 5+ 😢
thank you again for your time 🙏
Co-Authored-By: Oliver THEBAULT <Oliboy50@users.noreply.github.com>
CI seems to be passing now. Please take a look for further suggestions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you 🙆♂
Thanks for your time |
Tests are green and it seems to be working fine locally.
What could be other things we need to check?