Skip to content

Commit c7acc48

Browse files
authored
Merge pull request #264 from oliverklee/feature/cs-in-ci
Run the code sniffer in the CI pipeline
2 parents 40e9e89 + 862055b commit c7acc48

File tree

1 file changed

+18
-6
lines changed

1 file changed

+18
-6
lines changed

.github/workflows/ci.yml

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -79,21 +79,32 @@ jobs:
7979
run: |
8080
./vendor/bin/codacycoverage clover build/coverage/xml
8181
82-
phpstan:
83-
name: PHPStan
82+
static-analysis:
83+
name: Static Analysis
8484

8585
runs-on: ubuntu-20.04
8686

8787
needs: [ php-lint ]
8888

89+
strategy:
90+
fail-fast: false
91+
matrix:
92+
include:
93+
- command: sniff
94+
tool: phpcs
95+
php-version: 7.4
96+
- command: stan
97+
tool: phpstan
98+
php-version: 7.4
99+
89100
steps:
90101
- name: Checkout
91102
uses: actions/checkout@v2
92103

93104
- name: Install PHP
94105
uses: shivammathur/setup-php@v2
95106
with:
96-
php-version: 7.4
107+
php-version: ${{ matrix.php-version }}
97108
tools: "composer:v2, phive"
98109
coverage: none
99110

@@ -112,7 +123,8 @@ jobs:
112123
113124
- name: Install development tools
114125
run: |
115-
echo y | phive --no-progress update phpstan;
126+
# These workarounds are needed until PHIVE can do installs in a non-interactive way
127+
echo y | phive --no-progress update ${{ matrix.tool }};
116128
117-
- name: Run PHPStan
118-
run: "composer ci:php:stan"
129+
- name: Run Command
130+
run: composer ci:php:${{ matrix.command }}

0 commit comments

Comments
 (0)