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: #6591
Reviewed-By: cjihrig - Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
  • Loading branch information
Trott authored and evanlucas committed May 17, 2016
1 parent 341eaf2 commit 6806ebb
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 @@ -84,6 +84,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 6806ebb

Please sign in to comment.