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
feat: @PhpCsFixer
ruleset - normalize implicit backslahes in single quoted strings
#7965
base: master
Are you sure you want to change the base?
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
aebabe8
to
8fe7f1d
Compare
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
@PhpCsFixer
ruleset - normalize implicit backslahes in single quoted strings
8fe7f1d
to
cd21ad8
Compare
cd21ad8
to
02d96a9
Compare
Can this PR be approved? It is unambiguous that backslashes are commonly & significantly not quoted * and thus this the new config is the only one preferred. * here before #7786, Symfony, Lavarel, ... |
if you say this is Symfony best practice, add it to Sf ruleset and not Fixer's own one. |
@@ -119,7 +119,7 @@ public function getRules(): array | |||
'single_line_comment_style' => true, | |||
'single_line_empty_body' => true, | |||
'single_line_throw' => false, | |||
'string_implicit_backslashes' => ['single_quoted' => 'ignore'], | |||
'string_implicit_backslashes' => true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for changing any ruleset, remember to update ruleset integration test
as requested in #7786 (review)