Skip to content

Update for Laravel 5.5 #1280

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

Closed
wants to merge 1 commit into from
Closed

Update for Laravel 5.5 #1280

wants to merge 1 commit into from

Conversation

SimplyCorey
Copy link

This updates the belongsToMany signature for Laravel 5.5

See: https://laravel.com/docs/5.5/upgrade

@SimplyCorey SimplyCorey mentioned this pull request Aug 30, 2017
@SimplyCorey
Copy link
Author

Laravel support is going to need to be bumped to 5.5 in the composer.json file. Not sure how you want to handle that.

@RemiCollin
Copy link
Contributor

Nope, checks the composer.json, version constraint is ^5.1, so it will not prevent installing it from a 5.5 version.

What we might need is to tag a new package version (3.3 ?) to reflect the breaking change.

@akaroot
Copy link

akaroot commented Aug 31, 2017

see PR #1282

@SimplyCorey
Copy link
Author

SimplyCorey commented Aug 31, 2017

3.3 would also have to drop support for php5.6 in the test suite as laravel 5.5 no longer supports php <7.0

@SimplyCorey
Copy link
Author

Closing in favor of: #1300

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.

3 participants