This repository has been archived by the owner on Oct 4, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 28
feature_request(fix): remove leading spaces #155
Comments
Think you need to set the |
@nschonni, @gucong3000, + — eclint check doesn't show errors
- — eclint fix doesn't remove leading spaces in comments Is it possible remove leading spaces? They are not needed. Thanks. |
My comment was about comment blocks (😉 ) that were being flagged in your example. There are 3 editorconfig settings that deal specifically with comment blocks, that are separate from the |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1. Summary
It would be nice, if
eclint fix
will remove leading spaces for tabs users.2. Argumentation
Some tools automatically generate files with leading spaces.
eclint fix
doesn't fix it andeclint check
makes my builds failed.3. MCVE
.editorconfig
— I prefer tabs:KiraECLint.styl
— see Comment-Snippets issue for details:KiraECLint.js
— beginning of real file, that generate by grunt-modernizr:Leading spaces in
KiraECLint.styl
andKiraECLint.js
.4. Steps to reproduce
5. Expected behavior
Remove leading spaces after
fix
, successfulcheck
.6. Actual behavior
Thanks.
The text was updated successfully, but these errors were encountered: