From 83aa1f7f3f5722174c2c6a667631cc9977d34a81 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Wed, 4 May 2016 21:49:47 -0700 Subject: [PATCH] tools: lint for use of space in template strings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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: https://github.com/nodejs/node/pull/6591 Reviewed-By: cjihrig - Colin Ihrig Reviewed-By: Jeremiah Senkpiel Reviewed-By: Minwoo Jung Reviewed-By: Michaƫl Zasso --- .eslintrc | 1 + 1 file changed, 1 insertion(+) diff --git a/.eslintrc b/.eslintrc index 542be9ceaa..98a14b7ec8 100644 --- a/.eslintrc +++ b/.eslintrc @@ -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