Skip to content

Commit 30bc353

Browse files
committed
Add analyze-dependencies CI job
1 parent 45dfe26 commit 30bc353

File tree

1 file changed

+14
-9
lines changed

1 file changed

+14
-9
lines changed

.github/workflows/tests.yml

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,11 @@ jobs:
5858
description: 'with Coding Standards'
5959
coding-standards: true
6060

61+
# Dependencies
62+
- php-version: '8.4'
63+
description: 'with Analyze dependencies'
64+
analyze-dependencies: true
65+
6166
#Static Analysis (min PHP version)
6267
- php-version: '8.1'
6368
description: 'with Static Analysis'
@@ -134,28 +139,22 @@ jobs:
134139
yarn encore dev
135140
136141
- name: Fix Panther code
137-
if: matrix.coding-standards != true && matrix.static-analysis != true
142+
if: matrix.coding-standards != true && matrix.analyze-dependencies != true && matrix.static-analysis != true
138143
run: |
139144
sed -i 's/protected function doRequest(\$request)/protected function doRequest(object $request): object/' vendor/symfony/panther/src/Client.php
140145
141146
- name: Run PHPUnit
142-
if: matrix.coding-standards != true && matrix.static-analysis != true
147+
if: matrix.coding-standards != true && matrix.analyze-dependencies != true && matrix.static-analysis != true
143148
run: php vendor/phpunit/phpunit/phpunit
144149

145150
- name: Run Karma
146-
if: matrix.coding-standards != true && matrix.static-analysis != true
151+
if: matrix.coding-standards != true && matrix.analyze-dependencies != true && matrix.static-analysis != true
147152
run: xvfb-run ./node_modules/.bin/karma start --single-run
148153

149154
- name: Run PHP CS Fixer
150155
if: matrix.coding-standards
151156
run: php vendor/friendsofphp/php-cs-fixer/php-cs-fixer fix --diff --dry-run -v
152157

153-
- name: Analyze composer dependencies
154-
if: matrix.coding-standards
155-
run: |
156-
curl -LSs https://github.com/maglnet/ComposerRequireChecker/releases/latest/download/composer-require-checker.phar > composer-require-checker.phar
157-
php composer-require-checker.phar check composer.json
158-
159158
- name: Check license year
160159
if: matrix.coding-standards
161160
run: cat LICENSE |grep -E "\(c\) ([0-9]+\-)*`date +%Y`"
@@ -164,6 +163,12 @@ jobs:
164163
if: matrix.coding-standards
165164
run: ./node_modules/.bin/eslint assets/js/* tests/assets/js/*
166165

166+
- name: Analyze composer dependencies
167+
if: matrix.analyze-dependencies
168+
run: |
169+
curl -LSs https://github.com/maglnet/ComposerRequireChecker/releases/latest/download/composer-require-checker.phar > composer-require-checker.phar
170+
php composer-require-checker.phar check composer.json
171+
167172
- name: Run Psalm
168173
if: matrix.static-analysis
169174
run: php vendor/bin/psalm --stats --output-format=github

0 commit comments

Comments
 (0)