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

Add Symfony6 compatibility #41

Merged
merged 4 commits into from
Sep 29, 2022
Merged

Conversation

theofidry
Copy link
Contributor

Follow up of #40, adjusting the code based on the feedback.

@theofidry theofidry changed the title Ddebin/master Add Symfony6 compatibility Sep 28, 2022
@coveralls
Copy link

coveralls commented Sep 28, 2022

Coverage Status

Coverage decreased (-3.1%) to 89.916% when pulling 70e5f8d on theofidry:ddebin/master into 5ba0c7b on oradwell:master.

@theofidry
Copy link
Contributor Author

Side note: I think the project would benefit greatly from bumping dependencies. Trying to keep up with all dependencies is hard and I don't think it's worth it

@oradwell
Copy link
Owner

oradwell commented Sep 28, 2022

@theofidry thanks for the PR

you're right. I'm happy to drop support for Symfony 2.7, PHPUnit 6.x and PHP 7.0

@theofidry
Copy link
Contributor Author

Best to do it in a different PR though :)

@oradwell oradwell merged commit 1a52dc0 into oradwell:master Sep 29, 2022
@theofidry theofidry deleted the ddebin/master branch September 29, 2022 23:39
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.

4 participants