Skip to content

Commit b411f65

Browse files
author
Konrad Oboza
authored
[CI] Moved php-cs-fixer to another job
For more details see #271
1 parent 4be1233 commit b411f65

File tree

1 file changed

+31
-27
lines changed

1 file changed

+31
-27
lines changed

.github/workflows/ci.yaml

+31-27
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,31 @@ on:
88
pull_request: ~
99

1010
jobs:
11+
cs_fix:
12+
name: Run code style check
13+
runs-on: "ubuntu-20.04"
14+
strategy:
15+
matrix:
16+
php:
17+
- '8.0'
18+
steps:
19+
- uses: actions/checkout@v2
20+
21+
- name: Setup PHP Action
22+
uses: shivammathur/setup-php@v2
23+
with:
24+
php-version: ${{ matrix.php }}
25+
coverage: none
26+
extensions: 'pdo_sqlite, gd'
27+
tools: cs2pr
28+
29+
- uses: "ramsey/composer-install@v1"
30+
with:
31+
dependency-versions: "highest"
32+
33+
- name: Run code style check
34+
run: composer run-script check-cs -- --format=checkstyle | cs2pr
35+
1136
tests:
1237
name: Unit tests & SQLite integration tests
1338
runs-on: "ubuntu-20.04"
@@ -16,17 +41,11 @@ jobs:
1641
strategy:
1742
fail-fast: false
1843
matrix:
19-
experimental: [ false ]
2044
php:
2145
- '7.3'
2246
- '7.4'
23-
composer_options: [ "" ]
24-
include:
25-
- php: '8.0'
26-
composer_options: "--ignore-platform-req php"
27-
- php: '8.1'
28-
composer_options: "--ignore-platform-req php"
29-
skip_code_style: true
47+
- '8.0'
48+
- '8.1'
3049

3150
steps:
3251
- uses: actions/checkout@v2
@@ -42,18 +61,13 @@ jobs:
4261
- uses: "ramsey/composer-install@v1"
4362
with:
4463
dependency-versions: "highest"
45-
composer-options: "${{ matrix.composer_options }}"
4664

4765
- name: Setup problem matchers for PHPUnit
4866
run: echo "::add-matcher::${{ runner.tool_cache }}/phpunit.json"
4967

5068
- name: Run PHPStan analysis
5169
run: composer run-script phpstan
5270

53-
- name: Run code style check
54-
if: matrix.skip_code_style != true
55-
run: composer run-script check-cs -- --format=checkstyle | cs2pr
56-
5771
- name: Run unit test suite
5872
run: composer run-script unit
5973

@@ -86,12 +100,8 @@ jobs:
86100
php:
87101
- '7.3'
88102
- '7.4'
89-
composer_options: [ "" ]
90-
include:
91-
- php: '8.0'
92-
composer_options: "--ignore-platform-req php"
93-
- php: '8.1'
94-
composer_options: "--ignore-platform-req php"
103+
- '8.0'
104+
- '8.1'
95105

96106
steps:
97107
- uses: actions/checkout@v2
@@ -107,7 +117,6 @@ jobs:
107117
- uses: "ramsey/composer-install@v1"
108118
with:
109119
dependency-versions: "highest"
110-
composer-options: "${{ matrix.composer_options }}"
111120

112121
- name: Setup problem matchers for PHPUnit
113122
run: echo "::add-matcher::${{ runner.tool_cache }}/phpunit.json"
@@ -146,12 +155,8 @@ jobs:
146155
matrix:
147156
php:
148157
- '7.4'
149-
composer_options: [ "" ]
150-
include:
151-
- php: '8.0'
152-
composer_options: "--ignore-platform-req php"
153-
- php: '8.1'
154-
composer_options: "--ignore-platform-req php"
158+
- '8.0'
159+
- '8.1'
155160

156161
steps:
157162
- uses: actions/checkout@v2
@@ -167,7 +172,6 @@ jobs:
167172
- uses: "ramsey/composer-install@v1"
168173
with:
169174
dependency-versions: "highest"
170-
composer-options: "${{ matrix.composer_options }}"
171175

172176
- name: Setup problem matchers for PHPUnit
173177
run: echo "::add-matcher::${{ runner.tool_cache }}/phpunit.json"

0 commit comments

Comments
 (0)