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
Update wikimedia/less.php dependency #16976
Conversation
@@ -25,7 +25,7 @@ | |||
}, | |||
"config": { | |||
"platform": { | |||
"php": "7.2.5" | |||
"php": "7.2.9" |
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.
is this upgrade necessary? Does it mean we no longer support 7.2.5 as a minimum version @sgiehl ?
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.
That's the platform override value. So composer assumes 7.2.9 is installed, but the requirement is still set to 7.2.5
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.
👍
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.
@tsteur we maybe should merge this if possible before releasing a new version, as there might otherwise be issues on PHP 8
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.
@sgiehl planning to merge in a bit. https://github.com/wikimedia/less.php/blob/master/composer.json#L21 mentions it requires 7.2.9 so hoping it will still run on 7.2.5 (was changed in wikimedia/less.php@28baca4)
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.
Description:
We need to update wikimedia/less.php as the new release has some fixes for PHP 8.
I had to overwrite the composer platform config to PHP 7.2.9, as wikimedia/less.php requires PHP 7.2.9.
It should work with 7.2.5 as well, see wikimedia/less.php#50 (comment)
refs #16897
Review