Skip to content

Conversation

Chris53897
Copy link
Contributor

No description provided.

@Chris53897 Chris53897 changed the title chore: add tests for PHP 8.4 and 8.5 chore: add tests for PHP 8.4 Sep 15, 2025
@Hanmac
Copy link
Collaborator

Hanmac commented Sep 15, 2025

@Chris53897 the composer-unused as phar tool is problematic

The message is a lie:

Your Composer dependencies require a PHP version ">= 8.2.0". You are running (Version).

Because it isn't the composer dependencies, it's the phar itself that throwing the Error.

Code from the Phar:

if (!(PHP_VERSION_ID >= 80200)) {
    $issues[] = 'Your Composer dependencies require a PHP version ">= 8.2.0". You are running ' . PHP_VERSION . '.';
}

it would be better if the tool would be installed as dev dependency:
composer require --dev icanhazstring/composer-unused

@Hanmac
Copy link
Collaborator

Hanmac commented Sep 15, 2025

@loevgaard might need to look into this

@Chris53897
Copy link
Contributor Author

Chris53897 commented Sep 15, 2025 via email

@Hanmac
Copy link
Collaborator

Hanmac commented Sep 15, 2025

I will try to update the workflow with a different PR tomorrow

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