-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Drop PHP 5.X support #494
Drop PHP 5.X support #494
Conversation
62adc2a
to
6a752e5
Compare
6a752e5
to
cd5805b
Compare
cd5805b
to
88c34a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; Greg.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are are the other 7.xs dropped (no mention in the commit)?
Are there no later versions you want to test against? (Or does it just not work with any of the 8.x series?)
Fair point, for completeness - it's because 7.4 is the version of PHP 7 readily install-able on Rocky8 (see somewhat obscure reference in e2042aa)
8.X requires a later release of version 2.X of the Doctrine ORM before the tests will even run (279606c) |
Rebase after #509 is merged.This PR removes
composer-5.4.{json, lock}
tocomposer.{json.lock}
and drops PHP 5.x from the CI tests.