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

Update to Laravel 6.5 + PHPUnit 8.4 + PHP CS Fixer #85

Merged
merged 2 commits into from
Nov 5, 2019

Conversation

pionl
Copy link
Owner

@pionl pionl commented Nov 5, 2019

Original PR #80 by @TheFrankman and #84 @michaelaguiar

@pionl pionl merged commit 3801e6d into master Nov 5, 2019
This was referenced Nov 5, 2019
@nerg4l
Copy link

nerg4l commented Nov 5, 2019

Why did you added "illuminate/*": "5.1 - 6.5" instead of "illuminate/*": "5.1 - 5.8|^6.0"? Laravel moved to SemVer when it reached 6.0.

@pionl
Copy link
Owner Author

pionl commented Nov 5, 2019

Interesting point. I've followed the upgrade guide and there is no mention about using correct tag. Also I've used listed tags and it advised to use 6.5 (which I've installed in test repo and everything worked): https://github.com/pionl/laravel-chunk-upload-example/blob/master/composer.json

Maybe I can remove the strict to prevent updating the repo all the time. Thanks for the quick feedback.

@pionl pionl deleted the sharpstream-sharpstream branch November 5, 2019 21:28
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