Skip to content

Commit

Permalink
tools: lint for use of space in template strings
Browse files Browse the repository at this point in the history
There are over 70 files in the project using template strings and all of
them have followed the convention of no spaces in curly braces.

Good: `${foo}`

Not used: `${ foo }`

Since the project has adopted a convention, and ESLint has a rule to
enforce exactly this convention, enable the rule.

PR-URL: nodejs#6591
Reviewed-By: cjihrig - Colin Ihrig <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
  • Loading branch information
Trott committed May 8, 2016
1 parent 3f69ea5 commit 83aa1f7
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions .eslintrc
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,7 @@ rules:
no-new-symbol: 2
no-this-before-super: 2
prefer-const: 2
template-curly-spacing: 2

# Custom rules in tools/eslint-rules
align-function-arguments: 2
Expand Down

0 comments on commit 83aa1f7

Please sign in to comment.