Skip to content

Commit

Permalink
Make sure to build the phar only on php >= 8.2
Browse files Browse the repository at this point in the history
Since Box version 4.6.* requires min php 8.2 this had to be changed.
Since the Cap'n still has a min version of 8.0 the composer file has
to 'fake' the php version.
  • Loading branch information
sebastianfeldmann committed Aug 21, 2024
1 parent e9e9d13 commit b34d004
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ jobs:
strategy:
matrix:
php: ['8.0', '8.1', '8.2']
buildphar: [true]
buildphar: [false, false, true]
experimental: [false]
include:
- php: 8.3
buildphar: false
buildphar: true
experimental: false
env:
PHAR: build/phar/captainhook.phar
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ jobs:
with:
coverage: none
extensions: mbstring
php-version: 8.0
php-version: 8.2

- name: Install dependencies
run: composer install --prefer-dist --no-dev --no-interaction --no-progress
Expand Down
3 changes: 2 additions & 1 deletion composer.phar.json
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,8 @@
"sebastianfeldmann/captainhook": "*"
},
"config": {
"sort-packages": true
"sort-packages": true,
"platform": {"php": "8.0.0"}
},
"scripts": {
"post-install-cmd": "tools/phive install --force-accept-unsigned",
Expand Down

0 comments on commit b34d004

Please sign in to comment.