added support for laravel 9

[stojankukrika]

No description provided.

[ilzrv]

Thanks for contribution. Can you add Laravel 9 to test suite?

[stojankukrika]

I will try out :)

[stojankukrika]

@ilzrv added :)

[ilzrv]

Why did you make 2 PR? You should do it in one place.

[stojankukrika]

fixed

[stojankukrika]

But looks like tests also needed to be refactored...
https://github.com/stojankukrika/laravel-slow-query-detector/runs/6915560450?check_suite_focus=true
now package works with closure not with string anymore.

Can you help with it?

[ilzrv]

@stojankukrika You can try to change the version of this dependency like this "^1.6" || ^2.0

[stojankukrika]

It is like that, take a look.
I think tests need to be rewritten to use that new way (version 2. Old customers of the query detector package can use the old versions but new users can new version, that looks OK to me. What do you think?

[ilzrv]

I'll think about it. In the second version of the package, we can get away from supporting EOL versions of Laravel.

[stojankukrika]

Yes. Old version of this pagkage can use old versions of laravel and new versions of laravel can use a new version of package. Like this one laravel-datatables

[ilzrv]

@stojankukrika Completely updated the library, you can check PR: #4

[stojankukrika]

looks ok to me :)

[ilzrv]

Closed in favor of #4