PHP 8 Support

[coveralls]

Coverage increased (+0.01%) to 99.284% when pulling 0eaf9245148a63e208368ff951f0dab0639010c8 on blankse:php8 into c1536f66f4f8964018c061c1d61279c10d2724f8 on jkphl:master.

[jkphl]

Same here, trying to get this merged on the weekend

[blankse]

Dependency "clue/graph-composer" doesn't seem ready for 8.0 :(

[jkphl]

@blankse Unfortunately, simply removing clue/graph-composer is not an option for me. Need to get that PHP8 ready first.

[blankse]

@jkphl With the original clue/graph-composer it can be installed. Is this enough?

[jkphl]

@blankse Great news — that's a good start, yes. However, the one feature I added was the ability to skip development dependencies (or to only list development dependencies), as far as I remember. See https://github.com/clue/graph-composer/commit/28466d158dc678b946c757be129526cbf05ad3d7 and https://github.com/clue/graph-composer/commit/dff4bac16484fa429d6157e89118e922264323b5. We'd have to re-implement these features in order to move forward. Unfortunately, I won't have the capacity to do that anytime soon. Would you be able to step in here?

[rvanlaak]

If I'm correct clue/graph-composer is not a runtime dependency, but a "tool dependency" ?

Can we find a way to exclude this dependency from being a requirement for using micrometa in runtime? e.g. by running the clue/graph-composer via a Makefile task?

[blankse]

@jkphl I merged the original repository in your fork. See https://github.com/jkphl/graph-composer/pull/1 When you can merge this. I can change it back to your fork.

[blankse]

Ping @jkphl

[blankse]

@rvanlaak Can you create a release tag with this change?

[rvanlaak]

v3.1.0 just has been released

[blankse]

@rvanlaak Thank you 👍🏼