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

Restrict extra parentheses inside argument list of a function call/definition/etc. #988

Closed
wants to merge 1 commit into from

Conversation

astorije
Copy link
Member

Things like console.log( "Hello world" ); are now prohibited.

While I was playing with #977 and the dependency it refers to, I noticed we allow these for now (but thankfully we have none in-repo).

…finition/etc.

Things like `console.log( "Hello world" );` are now prohibited.
@astorije astorije added the Type: Feature Tickets that describe a desired feature or PRs that add them to the project. label Mar 28, 2017
@astorije astorije added this to the 2.3.0 milestone Mar 28, 2017
@astorije
Copy link
Member Author

Don't mind me, I'm useless.
#977 allows this indeed, but #975 that was created/merged after added this already 🤦‍♂️

@astorije astorije removed this from the 2.3.0 milestone Mar 28, 2017
@astorije astorije added the Status: Duplicate Tickets already solved/opened in the past. Should always link to the ticket they duplicate. label Mar 28, 2017
@astorije astorije closed this Mar 28, 2017
@astorije astorije deleted the astorije/eslint-no-space-in-parens branch March 28, 2017 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Duplicate Tickets already solved/opened in the past. Should always link to the ticket they duplicate. Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant