Support for PHP 8.0

[ildyria]

Currently Geocoder is only supporting php 7.2 and higher as seen in composer.json.

    "require": {
        "php": "^7.2",

Would it be possible to additionally enable support for php 8.0 which [will be/has been] released on November 26th, 2020 ?

[jbelien]

Excellent idea !
I'll test and make the update once PHP 8.0 is officially released (announced for later today).

[jbelien]

Apparently PHP 8.0 is not yet available in TravisCI : https://travis-ci.org/github/geocoder-php/Geocoder/builds/746552201
I'll try again in a few days!

[ildyria]

At LycheeOrg we got limited by the credits system on TravisCI (docker was taking a lot of time to build) so we migrated to Github actions. I have to say, this option is significantly faster than Travis builds.

[ildyria]

I spent a few hours setting up GitHub actions as it provides PHP 8.0 (https://github.com/ildyria/Geocoder/tree/add-github-actions)

Unfortunately, simply changing the composer file to also accept php 8.0 is not enough as some dependencies do not accept PHP 8: https://github.com/ildyria/Geocoder/runs/1470099431?check_suite_focus=true

[jbelien]

Awesome, thanks for the information @ildyria 👍

[jbelien]

Travis CI has been updated and now PHP 8.0 is available.
As you noticed, we need support for PHP 8.0 on php-http/curl-client (https://travis-ci.org/github/geocoder-php/Geocoder/jobs/746552203#L299).

[ildyria]

Yes, I know: php-http/curl-client#68

[ildyria]

Pull request on the Curl-client side: php-http/curl-client#71

[mruz]

Curl-client has been updated. @jbelien can you open a PR to check if php8 branch passes?

[jbelien]

I'm having a look this week 👍
I'll have to "upgrade" geocoder-php/provider-integration-tests. I'm working on it.

[jbelien]

PR is ready and waiting for review : geocoder-php/provider-integration-tests#17

[jbelien]

PR #1102 is ready and waiting for review 🚀