ReviewOS

also looking at this

meemalabs/laravel-media-converter

[1.0] Implement GitHub actions to run Pest test suite

#13
Merged chris wants to merge github-actions into main
3 files +14 -3

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/test.ymlmodified+6-5
Changes to .github/workflows/test.yml
@@ -36,14 +36,15 @@ jobs:
3636 AWS_IAM_ARN: ${{ secrets.AWS_IAM_ARN }}
3737 AWS_QUEUE_ARN: ${{ secrets.AWS_QUEUE_ARN }}
3838
39# - name: Cache dependencies
40# uses: actions/cache@v1
41# with:
42# path: ~/.composer/cache/files
43# key: dependencies-laravel-$-php-$-composer-$
39 - name: Cache dependencies
40 uses: actions/cache@v1
41 with:
42 path: ~/.composer/cache/files
43 key: dependencies-laravel-${{ matrix.laravel }}-php-${{ matrix.php }}-composer-${{ hashFiles('composer.json') }}
4444
4545 - name: Install dependencies
4646 run: |
47 composer require "orchestra/testbench:$" --no-interaction --no-update
4748 composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --no-suggest
4849
4950 - name: Create .env file