Skip to content

Commit

Permalink
chore(tests.yml): update dependencies installation
Browse files Browse the repository at this point in the history
- Added a new step to install dependencies
- Updated the composer command to require specific packages
- Added composer update command after package installation
  • Loading branch information
guanguans committed Aug 12, 2023
1 parent 3d62165 commit a3144d9
Showing 1 changed file with 14 additions and 10 deletions.
24 changes: 14 additions & 10 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ jobs:
test:
runs-on: ${{ matrix.os }}
strategy:
fail-fast: true
fail-fast: false
matrix:
os: [ ubuntu-latest ]
php: [ 7.4, 8.2 ]
Expand All @@ -22,18 +22,17 @@ jobs:
- laravel: 7.*
testbench: 5.*
exclude:
- laravel: 8.*
php: 8.0
dependency-version: prefer-lowest
- laravel: 8.*
- laravel: 7.*
php: 8.2
dependency-version: prefer-lowest
- laravel: 6.*
- laravel: 8.*
php: 8.2
- laravel: 9.*
php: 7.4
- laravel: 10.*
php: 8.0
php: 7.4

name: P${{ matrix.php }} - ${{ matrix.dependency-version }} - ${{ matrix.os }}
# name: P${{ matrix.php }} - ${{ matrix.dependency-version }} - ${{ matrix.os }}
name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }}

steps:
- name: Checkout code
Expand All @@ -52,8 +51,13 @@ jobs:
extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick
coverage: xdebug

# - name: Install dependencies
# run: composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --ansi -v

- name: Install dependencies
run: composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --ansi -v
run: |
composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" "nesbot/carbon:>=2.62.1" --no-interaction --no-update --ansi -v
composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --ansi -v
- name: Execute tests
run: composer test-coverage
Expand Down

0 comments on commit a3144d9

Please sign in to comment.