Add conflict with non-compatible versions of nikic/php-parser. by rodrigoaguilera · Pull Request #68 · beluga-php/docker-php · GitHub
Skip to content

Add conflict with non-compatible versions of nikic/php-parser.#68

Merged
flavioheleno merged 1 commit into
beluga-php:masterfrom
rodrigoaguilera:parser
Feb 21, 2024
Merged

Add conflict with non-compatible versions of nikic/php-parser.#68
flavioheleno merged 1 commit into
beluga-php:masterfrom
rodrigoaguilera:parser

Conversation

@rodrigoaguilera

Copy link
Copy Markdown
Collaborator

Bump psy/psysh to allow for nikic/php-parser v5.

Following up on the error from github actions
https://github.com/beluga-php/docker-php/actions/runs/7960004992/job/21769598590

I arrived with a google search to a similar error
sebastianbergmann/phpcov#113

One of the solutions was to bump nikic/php-parser so let's just do that sebastianbergmann/php-code-coverage@347a87a

it is strange that "prefer-lowest" is picking php-parser 4.10 since the requirements are at least for 4.18 🤷
This is the output of composer why nikic/php-parser

phpunit/php-code-coverage 9.2.30     requires  nikic/php-parser (^4.18 || ^5.0) 
psy/psysh                 v0.11.22   requires  nikic/php-parser (^4.0 || ^3.1)  
sebastian/complexity      2.0.3      requires  nikic/php-parser (^4.18 || ^5.0) 
sebastian/lines-of-code   1.0.4      requires  nikic/php-parser (^4.18 || ^5.0) 

For this reason I'm also bumping psysh, so the v5 of the parser can be installed.

Bump psy/psysh to allow for nikic/php-parser v5.
@sonarqubecloud

Copy link
Copy Markdown

@flavioheleno flavioheleno merged commit e2d4ec3 into beluga-php:master Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants