pdphilip / laravel-elasticsearch

Laravel Elasticsearch: An Elasticsearch implementation of Laravel's Eloquent ORM
https://elasticsearch.pdphilip.com/
MIT License
94 stars 17 forks source link

Refract/GitHub actions - Add Matrix Testing / Code Coverage #38

Closed use-the-fork closed 2 months ago

use-the-fork commented 2 months ago

Purpose of this PR:

This PR automates testing across different versions of PHP and Laravel to ensure compatibility and stability.

Key Updates:

image

image

pdphilip commented 2 months ago

Thanks @use-the-fork, I'll have a look here when I get a chance. Looking good though

pdphilip commented 2 months ago

This one should not have been closed; I pulled it into dev-main and then reversed it. Either way, I'd like to hold off on coverage for now. The nature of this package is challenging to cover, and even getting up to 90% would effectively be complete, but to the public 90% may look different and not offer the confidence we had hoped.

Happy to revisit this at a later point in time.