Skip to content
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

Lint rule: required-subscript-indentation #93

Open
wcjohnson opened this issue Sep 27, 2018 · 1 comment
Open

Lint rule: required-subscript-indentation #93

wcjohnson opened this issue Sep 27, 2018 · 1 comment

Comments

@wcjohnson
Copy link
Owner

Add a lint rule that enforces LSC 0.5.9-style subscript indentation:

// illegal
a
.b
// legal
a
  .b
@wcjohnson wcjohnson changed the title Lint rule: subscript indentation Lint rule: required-subscript-indentation Sep 27, 2018
@rattrayalex
Copy link

I'd be strongly in favor of this over #94

It'd be very confusing and frustrating to be writing this:

foo
.bar
.baz

and find this to do something completely different without throwing an error:

foo
.bar
['baz']

we should avoid that situation very strongly.

I'm also in favor of generally strong language style standards; I don't know of a reason not to recommend the indented style, which is more standard in the JS community (eg; prettier does this and I'm not aware of major requests to opt out of it).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants