search

shivammathur / setup-php

GitHub action to set up PHP with extensions, php.ini configuration, coverage drivers, and various tools.
https://setup-php.com
MIT License
2.89k stars 337 forks source link

Action takes 7 minutes to run #773

Closed szepeviktor closed 10 months ago

szepeviktor commented 10 months ago

Describe the bug

Version

Runners

Operating systems ubuntu-latest

PHP versions 8.1

To Reproduce https://github.com/mlocati/unipoints/blob/22602389f51f93b8e53721e6434f1262284c4cad/.github/workflows/tests.yml#L19-L35

Expected behavior Finish under 10 seconds

Screenshots/Logs https://github.com/mlocati/unipoints/actions/runs/6684760233/job/18162351050

Additional context PHIVE could download it as 1 file.

szepeviktor commented 10 months ago

⚠️ Not with version 2.25.5

https://github.com/shivammathur/setup-php/compare/2.25.5...2.26.0

I suspect the new Node.JS version.

shivammathur commented 10 months ago

@szepeviktor

Thanks for reporting this.

Yes, this is not related to php-cs-fixer, but the node process in v20 seems to get stuck after the script has run.

shivammathur commented 10 months ago

Fixed in https://github.com/shivammathur/setup-php/commit/9855c76987e3b3bbb80f4e19697b69126f9ad03a

szepeviktor commented 10 months ago

Thank you.

shivammathur commented 10 months ago

Released the fix in https://github.com/shivammathur/setup-php/releases/tag/2.27.0