Skip to content

Commit

Permalink
PHP 8.1: fix Phar building
Browse files Browse the repository at this point in the history
For the time being, until box-project/box#567 has been fixed:
1. Disable the addition of the requirement checker from Box as the version included in the last release of Box is not compatible with PHP 8.1.
2. Change the PHP version used to generate the Phar to PHP 8.0 to get round the problem the PHP compactor has with attributes.
    Note: the generated Phar file should still be compatible with all supported PHP versions.
  • Loading branch information
jrfnl committed Dec 1, 2021
1 parent 2dc7c7c commit daf0dfa
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 7.3
php-version: 8.0
extensions: exif, phar, openssl
coverage: none
ini-values: phar.readonly=Off
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 7.3
php-version: 8.0
extensions: exif, phar, openssl
coverage: none
ini-values: phar.readonly=Off, error_reporting=E_ALL, display_errors=On
Expand Down
1 change: 1 addition & 0 deletions box.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
"base-path": null,
"output": "parallel-lint.phar",
"chmod": "0755",
"check-requirements": false,
"compactors": [
"KevinGH\\Box\\Compactor\\Php"
],
Expand Down

0 comments on commit daf0dfa

Please sign in to comment.