Skip to content

Commit 0d5c35d

Browse files
committed
Rearrange CI jobs to run tests only after SA and CS checks
1 parent d07e12d commit 0d5c35d

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

.github/workflows/continuous_integration.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,12 +57,13 @@ jobs:
5757
with:
5858
dependency-versions: '${{ matrix.dependencies }}'
5959

60-
- name: Run PHPUnit
61-
run: vendor/bin/phpunit ${{ matrix.phpunit-args }}
62-
6360
- name: Validate composer
6461
run: composer validate --strict --no-check-lock
6562

63+
- name: Run php_codesniffer
64+
run: composer cs-check
65+
if: ${{ matrix.php-version == '7.2' }} # Do not suggest using features after 7.2
66+
6667
- name: phpstan-cache
6768
uses: actions/cache@v2
6869
with:
@@ -77,9 +78,8 @@ jobs:
7778
- name: Run phpstan
7879
run: composer phpstan
7980

80-
- name: Run php_codesniffer
81-
run: composer cs-check
82-
if: ${{ matrix.php-version == '7.2' }} # Do not suggest using features after 7.2
81+
- name: Run PHPUnit
82+
run: vendor/bin/phpunit ${{ matrix.phpunit-args }}
8383

8484
- name: Upload coverage results to Coveralls
8585
run: vendor/bin/php-coveralls --coverage_clover=coverage.xml -v

0 commit comments

Comments
 (0)