Skip to content

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