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.

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

Already on GitHub? Sign in to your account

Can braces_position fixer be implemented? #691

Open
ond-danny opened this issue Sep 2, 2024 · 0 comments
Open

Can braces_position fixer be implemented? #691

ond-danny opened this issue Sep 2, 2024 · 0 comments

Comments

@ond-danny
Copy link

Q A
Bug report? unsure
Feature request? yes
Library version 2.11.0

We are having issues aligning out phpinsights config to match our existing pint(1.16.2) config, which is based on the laravel preset.

The laravel config defines braces_position as braces is depreciated, and is configured to allow single line empty bodys, although this is contrary to PSR-12. (Opening brace must be on a new line).

No matter how I configure phpinsights it insists on producing code that fails the linter, despite the linter being the same CS-Fixer based.

Is it possible to get a new BracePositionFixer that reflects the current CSFixer ruleset?

Does anyone have any advise on how to get phpinsights to play well with pint in default config?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant