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

Allow phpunit 9 & update doctrine/inflect to ^2.0 #144

Merged
merged 1 commit into from
Feb 10, 2022

Conversation

DanielGSoftware
Copy link
Contributor

@DanielGSoftware DanielGSoftware commented Feb 9, 2022

Our company wants to upgrade to Laravel 9, but certain dependencies need to be allowed.

  • Doctrine 1.4 and 2.0 are not compatible with each other, so I set the minimum version to ^2.0. This will automatically make the minimum php version required to 7.2. I went through the code and fixed the breaking changes.

  • We also need to allow php-unit 9, but this is not compatible with phpunit 5 and is from version 6 or higher. Also fixed all breaking changes.

Tests succeed: DanielGSoftware#1

@goetas goetas merged commit 33814ba into goetas-webservices:master Feb 10, 2022
@goetas
Copy link
Member

goetas commented Feb 10, 2022

thanks a lot for this 🎉

@DanielGSoftware
Copy link
Contributor Author

DanielGSoftware commented Feb 10, 2022

@goetas You're welcome, and thank you for this package :)! Can you also make a new tag for this? (I think we need to tag a 1.0.0 version because of dropped support for php 7.1)

I also noticed that CI isn't working properly if I'm correct, if you want to switch over to Github actions, I can make a PR with a script if you'd like

@goetas
Copy link
Member

goetas commented Feb 10, 2022

having github actions would be great! thanks a lot

I will create a new tag, dropping dependencies is not a BC break, so a usual increment is ok

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.

None yet

2 participants