Support PHP 8.0

[alexislefebvre]

Allow PHP8

[alexislefebvre]

It will require an update from https://github.com/nelmio/alice

See nelmio/alice#1064

[nicodmf]

@theofidry any news on this ?

[alexislefebvre]

This PR is blocked by a PR that is blocked by another PR. 😅

[Yozhef]

@alexislefebvre nelmio/alice#1064 is done =)

[alexislefebvre]

It is blocked by theofidry/composer-inheritance-plugin#8 now.

[alexislefebvre]

https://github.com/doctrine/mongodb-odm doesn't support PHP 8 yet (PHP 8 is allowed on master branch but it hasn't been released yet).

[VincentLanglet]

Can't we skip temporary the mongo-related test on php8 ?
Doctrine/orm supports php8, it should be allowed to use this package on php8 if we don't use mongo.

[alexislefebvre]

I thought about that, and I think it would be confusing to publish a new release that may break users' tests (if they didn't locked their dependencies, that's a good practice but some users don't do that).

To avoid this, a new release which declare conflicts with the Mongo packages could be published.

[VincentLanglet]

I thought about that, and I think it would be confusing to publish a new release that may break users' tests (if they didn't locked their dependencies, that's a good practice but some users don't do that).

Why will it break the users tests ?
You're just allowing php8 in this PR.

User without mongo, will be able to upgrade to PHP8, user with mongo won't be able to upgrade to PHP8 ; not because of us, but because of mongo which doesn't currently support PHP8.

There is no change in the code.
If someone use this library on PHP7.4 with mongo, it will still work the same ; proof => test are green on 7.4 with mongo.

[alexislefebvre]

@VincentLanglet you're totally right, I overthought that. 🙂

[stof]

The composer-inheritance-plugin is now solved, so this work should be resumed

[alexislefebvre]

One job looks dead but the PHP8 job is green. Feel free to review the changes.

[theofidry]

Thank you!