Skip to content

/standard

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

New rule: Disallow spaces inside of computed properties (computed-property-spacing) #1315

Closed
feross opened this issue Jul 5, 2019 · 1 comment

Comments

@feross
Copy link
Member

commented Jul 5, 2019

https://eslint.org/docs/rules/computed-property-spacing

Replace our custom computed-property-even-spacing rule from eslint-plugin-standard with the eslint rule that just disallows spaces

@feross feross added this to the standard v13 milestone Jul 5, 2019

@feross feross added accepted enhancement labels Jul 5, 2019

@feross

This comment has been minimized.

Sign in to view
Copy link
Member Author

commented Jul 5, 2019

Ecosystem impact: only one package effected.

Definitely should do this to be consistent with the array bracket rule that will be in standard 13.

@feross feross closed this Jul 6, 2019

@feross feross referenced this issue Jul 6, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accepted enhancement
Milestone
  standard 13
1 participant
@feross
You can’t perform that action at this time.