laminas-ci-matrix-action
laminas-ci-matrix-action copied to clipboard
Composer `php` constraints with `,` are not supported
Bug Report
Q | A |
---|---|
Version(s) | 1.11.5 |
Summary
When a component uses a more complex constraint which contains ,
, the matrix is not able to infer supported PHP versions anymore.
Current behavior
Supported PHP versions are not detected at all.
How to reproduce
composer.json
{
"require": {
"php": ">=5.6,<=8.1.99"
}
}
Expected behavior
All versions between PHP 5.6 and 8.1.99 are detected. As of writing this issue, this would be:
- 5.6
- 7.0
- 7.1
- 7.2
- 7.3
- 7.4
- 8.0
- 8.1
Would a better way not be to just replace ,
with ||
or a space?
@internalsystemerror ,
means AND
, while ||
means OR
:thinking:
@Ocramius I've just been reviewing the semver guide on npm https://www.npmjs.com/package/semver and I believe ,
->
(space) would be the only transformation necessary