feat: add no-multiple-whitespace rule#370
Conversation
|
Hello @Ericlm, thanks for this patch. It seems working; any plans to release it? |
|
Any updates on this? would be nice to have 😄 |
|
@francoismassart What do you think of this rule? |
|
This rule is needed. can't wait :) |
|
very needed rule thanks for your efforts @Ericlm |
|
Hi, e.g. ctl(`
sm:w-6
w-8
container
w-12
flex
lg:w-4
`);Maybe you should detect if the current classlist contains new lines or carriages return and handle theses cases. |
I handled the case by returning if the classNames contains any line break/carriage return. It won't remove spaces around classes in this syntax, as it may contains leading whitespaces. At best, it could remove trailing whitespaces I think. 🤔 |
Whitespace automatic removal
Description
This PR implements a new rule called
no-multiple-whitespacewhich removes unnecessary whitespaces between classes.Closes #263
Type of change
How Has This Been Tested?
I added a corresponding
no-multiple-whitespacetest file.Test Configuration:
Checklist: