-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Remove excess spacing before multiline block %}
#245
Comments
Hi, the main reason is that all the spacing rules are ignoring new lines, because there is nothing about it in the twig standard. People might prefer:
over
cf The AbstractSpacingRule is used
I'm sure you won't exemple
to be fixed to We could consider adding an option to the DelimiterSpacingRule (because it's maybe the only one where it make sens to remove newlines ?). Which means adding an option (in the construct of AbstractSpacingRule with) to skip or not newlines. Wdyt ? |
I agree that this might be a preference that shouldn't be enabled by default. So I'm all ears for introducing a new option that defaults to the current behavior. |
I'll be at least one week off so won't be able to help on this soon. But if you want to try
will certainly be And test will be necessary. |
I tried something in #247 but I couldn't get it to work. |
Rule(s) related to or rule(s) missing
Using the standard TwigCsFixer rules I notice something odd.
Actual behavior
Expected behavior
Why isn't the excess newline and spacing not removed?
The text was updated successfully, but these errors were encountered: