-
Notifications
You must be signed in to change notification settings - Fork 19
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
Laravel 10 pakcage conflict #52
Comments
@thevilledev What do you suggest? I will create a PR accordingly. |
Did you test upcloud-php-api with symfony/serializer@6.2? Was it working ok? If yes, then I think we could simply bump the version in composer.json. So change that to Or we could make it even more relaxed, like |
I have created a fork for PHP 8.1 only to get it working
Also, need some changes in the following files also
However, I was unable to make it work for earlier PHP versions, so I didn't create a pull request. Please refer to the changes here - master...sandeshjangam:upcloud-php-api:master |
Seems like the package update would work with I Added you @sandeshjangam as a co-author to the related commit in that PR as it was not obvious which commits to cherry-pick from your fork so I copied the code changes manually. |
Hello, this should now be resolved in v3.0.0. |
I'll close this as completed, please re-open or create a new issue, if there are still issues when installing together with Laravel. |
While installing this apckage with Laravel 10 with PHP 8.1
Your requirements could not be resolved to an installable set of packages.
Conflict is related to this package. If I update this to 6.2 it causes problems to other PHP version.
symfony/serializer
package conflictThe text was updated successfully, but these errors were encountered: