matchish / laravel-scout-elasticsearch

Search among multiple models with ElasticSearch and Laravel Scout
MIT License
702 stars 113 forks source link

php 8.2 added for the tests #238

Closed hkulekci closed 1 year ago

hkulekci commented 1 year ago

I think we need to test for php8.2, too.

codecov-commenter commented 1 year ago

Codecov Report

Base: 96.05% // Head: 96.05% // No change to project coverage :thumbsup:

Coverage data is based on head (b048460) compared to base (7b9fc92). Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #238 +/- ## ========================================= Coverage 96.05% 96.05% Complexity 190 190 ========================================= Files 36 36 Lines 634 634 ========================================= Hits 609 609 Misses 25 25 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Sergey+Shlyakhov). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Sergey+Shlyakhov)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

matchish commented 1 year ago

I like tests) Should I merge it now?

hkulekci commented 1 year ago

I think we need to merge this.