Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

Add / to punctuators that trip up ASI #12169

Closed
wants to merge 1 commit into from
Closed

Add / to punctuators that trip up ASI #12169

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Apr 1, 2016

Starting a line with / will make ASI think it's a division not a regex literal (although starting a line with / is rare).

Starting a line with `/` will make ASI think it's a division not a regex literal (although starting a line with `/` is rare).
@zkat
Copy link
Contributor

zkat commented Apr 6, 2016

Ever since npm@3, we've been using standard as our linter, and a nice big chunk of this style guide is basically absorbed into that, and its various rules. This page needs a lot more done and it might be worth removing entirely. :\ cc @ashleygwilliams for doc issue

@iarna
Copy link
Contributor

iarna commented Jun 30, 2017

We're closing this pull request as it has gone sixty days without activity. In our experience pull requests that go this long are unlikely to land. If you have updates to it, please feel free to open a new pull request.

For more information about our new issue aging policies and why we've instituted them please see our blog post.

@iarna iarna closed this Jun 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants