Skip to content

Commit

Permalink
keep the older test names but override the constraint ...
Browse files Browse the repository at this point in the history
keep the names because we have required checks based on the names. PR checks would need to be re-triggered
  • Loading branch information
jasonvarga committed Dec 8, 2020
1 parent a785ae2 commit 6451c97
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,13 @@ jobs:
matrix:
php: [7.4, 7.3, 7.2, 8.0]
composer: [1, 2]
laravel: [^8.18.1, 7.*, 6.*]
laravel: [8.*, 7.*, 6.*]
dependency-version: [prefer-lowest, prefer-stable]
os: [ubuntu-20.04]
include:
- laravel: ^8.18.1
- laravel: 8.*
laravel-constraint: ^8.18.1
- laravel: 8.*
testbench: 6.*
- laravel: 7.*
testbench: 5.*
Expand All @@ -32,7 +34,7 @@ jobs:
dependency-version: prefer-lowest
additional-deps: '"mockery/mockery:>=1.3.3"'
exclude:
- laravel: ^8.18.1
- laravel: 8.*
php: 7.2

name: P${{ matrix.php }} - C${{ matrix.composer }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }} - ubuntu-latest
Expand All @@ -51,7 +53,7 @@ jobs:

- name: Install dependencies
run: |
composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" ${{ matrix.additional-deps }} --no-interaction --no-update
composer require "laravel/framework:${{ matrix.laravel-constraint || matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" ${{ matrix.additional-deps }} --no-interaction --no-update
composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --no-suggest
- name: Execute tests
Expand Down

0 comments on commit 6451c97

Please sign in to comment.