Skip to content

Commit 80a8dcf

Browse files
committed
minor #6949 Reorganize linters in GitHub CI actions (javiereguiluz)
This PR was squashed before being merged into the 4.x branch. Discussion ---------- Reorganize linters in GitHub CI actions We're going to add a linter/formatter for CSS/JS ... so let's reorganize things a bit first. Commits ------- 0be66bc Reorganize linters in GitHub CI actions
2 parents 6049ea8 + 0be66bc commit 80a8dcf

File tree

1 file changed

+42
-33
lines changed

1 file changed

+42
-33
lines changed

.github/workflows/ci.yaml

Lines changed: 42 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -6,39 +6,6 @@ on:
66
pull_request:
77

88
jobs:
9-
linter:
10-
runs-on: ubuntu-latest
11-
steps:
12-
- uses: actions/checkout@v4
13-
14-
- name: Setup PHP, with composer and extensions
15-
uses: shivammathur/setup-php@v2
16-
with:
17-
php-version: 8.2
18-
coverage: none
19-
20-
- name: Install dependencies
21-
run: |
22-
composer install
23-
vendor/bin/simple-phpunit install
24-
25-
- name: DOCtor-RST
26-
uses: docker://oskarstark/doctor-rst
27-
with:
28-
args: --short
29-
env:
30-
DOCS_DIR: "doc/"
31-
32-
# see https://github.com/OskarStark/php-cs-fixer-ga
33-
- name: PHP-CS-Fixer
34-
uses: docker://oskarstark/php-cs-fixer-ga
35-
with:
36-
args: --diff --dry-run
37-
38-
- name: Lint logical CSS properties
39-
run: |
40-
php ./src/Resources/bin/logical-css-properties-linter.php
41-
429
phpstan:
4310
runs-on: ubuntu-latest
4411
steps:
@@ -159,3 +126,45 @@ jobs:
159126
SYMFONY_DEPRECATIONS_HELPER: "max[total]=999999&ignoreFile=./tests/baseline-ignore.txt"
160127
run: |
161128
./vendor/bin/simple-phpunit tests/Controller/PrettyUrls/PrettyUrlsControllerTest.php
129+
130+
php-linter:
131+
runs-on: ubuntu-latest
132+
steps:
133+
- uses: actions/checkout@v4
134+
135+
# see https://github.com/OskarStark/php-cs-fixer-ga
136+
- name: PHP-CS-Fixer
137+
uses: docker://oskarstark/php-cs-fixer-ga
138+
with:
139+
args: --diff --dry-run
140+
141+
doc-linter:
142+
runs-on: ubuntu-latest
143+
steps:
144+
- uses: actions/checkout@v4
145+
146+
- name: DOCtor-RST
147+
uses: docker://oskarstark/doctor-rst
148+
with:
149+
args: --short
150+
env:
151+
DOCS_DIR: "doc/"
152+
153+
css-linter:
154+
runs-on: ubuntu-latest
155+
steps:
156+
- uses: actions/checkout@v4
157+
158+
- name: Setup PHP, with composer and extensions
159+
uses: shivammathur/setup-php@v2
160+
with:
161+
php-version: 8.2
162+
coverage: none
163+
164+
- name: Install dependencies
165+
run: |
166+
composer install
167+
168+
- name: Lint logical CSS properties
169+
run: |
170+
php ./src/Resources/bin/logical-css-properties-linter.php

0 commit comments

Comments
 (0)