Skip to content

Commit 5168e81

Browse files
ci: Add cs2pr handling for php-cs-fixer; avoid doing composer install (#814)
This PR uses the tools available from the [setup-php ](https://github.com/marketplace/actions/setup-php-action) action to improve the feedback to better readable for humans using [cs2pr](https://github.com/staabm/annotate-pull-request-from-checkstyle) and avoids running `composer install` during the flow.
1 parent 528b39f commit 5168e81

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

.github/workflows/style-check.yml

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,8 @@ on:
88
branches:
99
- master
1010

11-
env:
12-
COMPOSER_FLAGS: "--ansi --no-interaction --no-progress --prefer-dist"
13-
1411
jobs:
15-
tests:
12+
style-check:
1613
name: "Style Check"
1714

1815
runs-on: ubuntu-latest
@@ -28,10 +25,8 @@ jobs:
2825
extensions: "intl, zip"
2926
ini-values: "memory_limit=-1, phar.readonly=0, error_reporting=E_ALL, display_errors=On"
3027
php-version: "7.4"
31-
tools: composer
32-
33-
- name: "Update dependencies"
34-
run: "composer update ${{ env.COMPOSER_FLAGS }}"
28+
tools: cs2pr, php-cs-fixer:3.3
3529

3630
- name: "Run style-check"
37-
run: "composer style-check"
31+
run: |
32+
composer style-check -- --format=checkstyle src | cs2pr

CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1111
### Fixed
1212
- Use parallel-lint and cs2pr for improved feedback on linting errors ([#812](https://github.com/jsonrainbow/json-schema/pull/812))
1313
- Array with number values with mathematical equality are considered valid ([#813](https://github.com/jsonrainbow/json-schema/pull/813))
14-
## Changed
14+
### Changed
1515
- Correct PHPStan findings in validator ([#808](https://github.com/jsonrainbow/json-schema/pull/808))
16+
- Add cs2pr handling for php-cs-fixer; avoid doing composer install ([#814](https://github.com/jsonrainbow/json-schema/pull/814))
1617

1718
## [6.3.1] - 2025-03-18
1819
### Fixed

0 commit comments

Comments
 (0)