notslang / editorconfig-tools

Tools for verifying/fixing code style based on an EditorConfig file
https://npmjs.com/package/editorconfig-tools
GNU General Public License v3.0
54 stars 6 forks source link

indent_style=space doesn't give warning when a single line uses tab #36

Open SimenB opened 7 years ago

SimenB commented 7 years ago

Discovered in airbnb/javascript#1102, using editorconfig-tools check.

It seems like the check doesn't go through all lines. If I use indent_style=space, a single tab goes unnoticed, but if all lines are using tabs, I get yelled at. It's the same the other way around; having indent_style=tab, a single line using spaces for indentation goes unnoticed, but if the whole file is indented with spaces, it fails.