Skip to content

TASK: Introduce PSR12 linting via phpcs and autofix files - #4002

Draft
mhsdesign wants to merge 1 commit into
9.0from
task/introduce-psr12-linting
Draft

mhsdesign wants to merge 1 commit into
9.0from
task/introduce-psr12-linting

Conversation

@mhsdesign

Copy link
Copy Markdown
Member

What I did

How I did it

How to verify it

@Sebobo

Sebobo commented Oct 6, 2025

Copy link
Copy Markdown
Member

StyleCI has 2 complaints. Are those from the upmerge or did phpcs ignore those?

@Sebobo
Sebobo self-requested a review October 6, 2025 11:17
@mhsdesign

Copy link
Copy Markdown
Member Author

no this fix would also replace StyleCi and its riddles ;) Ill see if the reasons are legit...

Okay style CI likes no space after an anonymous class return new class() {} but phpcs wants it return new class () {} so they fight each other ...

befor continuing with this well need to see what all our code in Neos.Neos and co says as we only use phpcs partly ... but the style CI revolution should be started there as well..

@mhsdesign
mhsdesign marked this pull request as draft October 6, 2025 12:25

This branch has not been deployed

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants