rollbar / rollbar-php-laravel

Rollbar error monitoring integration for Laravel projects
https://docs.rollbar.com/docs/laravel
140 stars 39 forks source link

Added support for the illuminate/support package to also accept version 11 #157

Closed GertjanRoke closed 6 months ago

GertjanRoke commented 6 months ago

With this merge request, it will add support for Laravel 11. The only change that is needed for supporting Laravel 11 is allowing the Illuminate package to accept version 11.0 or higher.

I made this MR because I was already working with Laravel 11 (dev-master) and wanted to install this package as well, but was running into the issue that it's not supporting version 11. After bumping the version inside a local installation, I ran all error logging command versions, and they were all added to Rollbar issue console.

Don't know if you wanted to keep that PR template boilerplate, if so please let me know I can add it again :)

GertjanRoke commented 6 months ago

@zdavis-rollbar do you have any idea when this will be tagged?

zdavis-rollbar commented 6 months ago

Tagged and released.

GertjanRoke commented 6 months ago

@zdavis-rollbar maybe a strange question, but I guess the tagging went wrong because you made it like this: V8.0.01. But Packagist didn't pick this up, I guess because it's an incorrect format. It should be something like this V8.0.1 instead.

You can see it here as well, it's not in the list of tags: https://packagist.org/packages/rollbar/rollbar-laravel

zdavis-rollbar commented 6 months ago

I see packagist wasn't auto updated. I have updated packagist and you should be able to get access to it now. I also renamed the tag to v8.0.1