clutchski / coffeelint

Lint your CoffeeScript.
http://www.coffeelint.org
Other
1.18k stars 171 forks source link

Tabs indented literate code fixed #561

Closed KrysKruk closed 8 years ago

KrysKruk commented 8 years ago

In the previous PR #557 I forgot that \s in RegExp matches also tab characters. Right now lines with 4 and more tabs occurs in unexpected indentation code, because first four tabs are removing. This PR fixes this bug.

swang commented 8 years ago

Merged in 5623ac19f71c07d2dc2deb3dc9c3a8e64355dc2f

KrysKruk commented 8 years ago

Thank you for merging this. I appreciate it. Also I want to apologise for no response in the proper time. I agree with your suggestions. I'll try to prepare PRs for them. They are both style issues, so IMHO not so important for the NPM release.

swang commented 8 years ago

Hi. I went ahead and made the changes and just merged them into your commit. No need to worry about making the changes.