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

Don't require spaces inside expression parentheses #134

Open
gibson042 opened this issue Dec 30, 2015 · 2 comments
Open

Don't require spaces inside expression parentheses #134

gibson042 opened this issue Dec 30, 2015 · 2 comments

Comments

@gibson042
Copy link
Member

Excluding spaces from inside expression parentheses, especially in loop-statement assignments, has historically been common in jQuery—and used intentionally to visually distinguish expression parentheses from statement parentheses—but was prohibited by jscs-dev/node-jscs@0f78baf in response to #104 (which was removing our complicated function-call exceptions).

Since that time, JSCS has added an ignoreParenthesizedExpression option to requireSpacesInsideParentheses. We should adopt it and update the style guide accordingly.

@kborchers
Copy link
Member

👍 for what it's worth from someone who rarely gets to write code anymore 😄

@dmethvin
Copy link
Member

I'm strongly for this as well.
cm87kwq

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

No branches or pull requests

3 participants