From 6806ebb608f8e1dc6a6715e141825f32571ef95e 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 757024f89c6262..14fbc039a0d802 100644 --- a/.eslintrc +++ b/.eslintrc @@ -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