-
Notifications
You must be signed in to change notification settings - Fork 38
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
Slim 4 #47
Conversation
Just a heads up @mnapoli there will be upcoming breaking changes to I know, there’s already a lot of BC changes but unfortunately we have to do this so See slimphp/Slim#2780 for more information |
I'm putting this on hold waiting for slimphp/Slim#2793 @l0gicgate I'm commenting in the issue you linked to. |
@mnapoli I think we’re in the clear here. Just released 4.2 today and we avoided all potential BCs thankfully! |
@l0gicgate awesome, thank you for letting me know! |
cannot wait for slim 4 support |
Is this ready to go? Is there a way to use this via composer so I can try it out on some existing projects I would like to upgrade to Slim 4? |
Any update on this? Definitely interested in testing this out on Slim 4. |
I have updated the issue description with a TODO list of what's left to do. I have loads of work this week, I hope to get on this next week. |
Hi @mnapoli Thanks for the update! I've created a new PR that solves the two todo items you mentioned here. |
I have opened a PR for the documentation as well: PHP-DI/PHP-DI#682 I'll merge everything, tag, release and deploy the new docs. Thanks everyone for the help! |
I think, with this merged PHP-DI/PHP-DI#680 can be closed, too. |
@SvenRtbg yes thanks! |
Originally #46
resolves #22
TODO: