jolicode / elastically

🔍 JoliCode's Elastica wrapper to bootstrap Elasticsearch PHP integrations
248 stars 37 forks source link

Test CI against PHP 8.1 & fix deprecation #98

Closed jmsche closed 2 years ago

lyrixx commented 2 years ago

This super cool !

I'm wondering: Do we really need all the combinaisons in the matrix? ATM, the matrix is quite big and burn some energy for nothing.

IMHO the following matrix is enough:

PHP 7.4 + SF 4.4 + lower deps PHP 7.4 + SF 5.4 PHP 8.0 + SF 6 PHP 8.1 + SF 6

What do you think ?

jmsche commented 2 years ago

Okay, I'm going to adapt it :)

jmsche commented 2 years ago

Ready for review 👍

lyrixx commented 2 years ago

Thanks