Skip to content

Commit 6060b96

Browse files
committed
Properly run phpstan in ci
1 parent a6e2909 commit 6060b96

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

.github/workflows/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,4 +48,4 @@ jobs:
4848
run: bash ci/run_tests_dockerized.sh ${{ matrix.PHP_VERSION }}
4949

5050
- name: Run static analysis (PHP 7.2 only)
51-
run: if [[ "$STATIC_ANALYSIS" == true ]]; then bash ci/run_phpstan_dockerized.sh ${{ matrix.PHP_VERSION }}; fi
51+
run: if [[ "${{ matrix.STATIC_ANALYSIS }}" == true ]]; then bash ci/run_phpstan_dockerized.sh ${{ matrix.PHP_VERSION }}; fi

ci/run_phpstan.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,6 @@ set -xeu
66
# This installs and runs phpstan
77
# TODO: Remove separate phpstan install step after https://github.com/microsoft/tolerant-php-parser/pull/385 is merged
88
if [ ! -d vendor/phpstan/phpstan ]; then
9-
composer.phar require --dev phpstan/phpstan@^1.8
9+
composer.phar require --dev phpstan/phpstan=^1.8
1010
fi
1111
./vendor/bin/phpstan analyze

0 commit comments

Comments
 (0)