-
Notifications
You must be signed in to change notification settings - Fork 40
Add Laravel 8 support #103
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
Conversation
Laravel 8 is released today .. how are we doing on getting this reviewed, merged and released? @bastien-phi @waltjones @coryvirok @ArturMoczulski @emilsundberg @brianr @mrunalk @icsahn-rollbar |
@bastien-phi @waltjones @coryvirok @ArturMoczulski @emilsundberg @brianr @mrunalk @icsahn-rollbar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good to me. Adding 'prefer-stable' will make sure to use more stable packages than 'dev' if available. https://getcomposer.org/doc/04-schema.md#minimum-stability
@letaniaferreira Great. Can this be merged and a new version be released asap? Currently this is preventing users of this lib from upgrading to Laravel 8.x. |
I will most likely have this out to you by tomorrow @jonnott. I am currently trying to get the right privileges to complete the version update process. |
Thanks @letaniaferreira! |
Hi @fgilio, @jonnott , @bastien-phi, @ElRochito and @dacoto97, we're still working to get this to Packagist. Did the merge already got you into good shape? |
Getting it onto packagist.org would be 👍 |
Should be good to go: https://packagist.org/packages/rollbar/rollbar-laravel |
Fixes #102
I tested this locally. It looks like no other change is needed for Laravel 8