Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CI: Use PHPStan's native Github error formatter #7487

Merged
merged 1 commit into from Dec 1, 2023
Merged

CI: Use PHPStan's native Github error formatter #7487

merged 1 commit into from Dec 1, 2023

Conversation

Wirone
Copy link
Member

@Wirone Wirone commented Dec 1, 2023 •
edited

Using cs2pr is not required anymore in terms of PHPStan 馃檪.

There's no violations in CI currently, but the output looks like this (modified file locally to achieve this):
image

@Wirone Wirone requested a review from keradus December 1, 2023 10:43
@Wirone Wirone self-assigned this Dec 1, 2023
@coveralls
Copy link

Coverage Status

coverage: 94.918%. remained the same
when pulling 3321354 on Wirone:codito/phpstan-github-formatter
into 81d5d9e on PHP-CS-Fixer:master.

@keradus keradus merged commit d680159 into PHP-CS-Fixer:master Dec 1, 2023
25 of 26 checks passed
@Wirone Wirone deleted the codito/phpstan-github-formatter branch December 1, 2023 12:01
@ondrejmirtes
Copy link

You don't need even to choose GitHub, it gets autodetected ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@keradus keradus keradus approved these changes

Assignees

@Wirone Wirone

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Wirone @coveralls @ondrejmirtes @keradus