ReviewOS

also looking at this

meemalabs/laravel-meema

Implement Tests

#8
Closed glennmichael123 wants to merge implemen-test into main
6 files +16 -18

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

.github/workflows/run-tests.ymlmodified+4-5
Changes to .github/workflows/run-tests.yml
@@ -9,16 +9,14 @@ jobs:
99 fail-fast: true
1010 matrix:
1111 os: [ubuntu-latest, windows-latest]
12 php: [7.2, 7.3, 7.4, 8.0]
13 laravel: [6.*, 7.*, 8.*]
12 php: [7.3, 7.4, 8.0]
13 laravel: [7.*, 8.*]
1414 dependency-version: [prefer-stable]
1515 include:
1616 - laravel: 8.*
1717 testbench: 6.*
1818 - laravel: 7.*
1919 testbench: 5.*
20 - laravel: 6.*
21 testbench: 4.*
2220
2321 name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }} - ${{ matrix.os }}
2422
@@ -41,7 +39,8 @@ jobs:
4139
4240 - name: Install dependencies
4341 run: |
44 composer install
42 composer require "laravel/framework:$" "orchestra/testbench:$" --no-interaction --no-update
43 composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --no-suggest
4544
4645 - name: Execute tests
4746 run: vendor/bin/pest