From 643ec240bd901295d9e9ea5c43fc20109c33e982 Mon Sep 17 00:00:00 2001 From: Brad Zacher Date: Sun, 10 May 2020 14:16:31 -0700 Subject: [PATCH] feat(eslint-plugin): [restrict-template-expressions] `allowNumber: true` by default (#2005) --- .../docs/rules/restrict-template-expressions.md | 2 +- .../src/rules/restrict-template-expressions.ts | 6 +++++- .../tests/rules/restrict-template-expressions.test.ts | 2 ++ 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/packages/eslint-plugin/docs/rules/restrict-template-expressions.md b/packages/eslint-plugin/docs/rules/restrict-template-expressions.md index 4bb2a062ad9..260a8eb188d 100644 --- a/packages/eslint-plugin/docs/rules/restrict-template-expressions.md +++ b/packages/eslint-plugin/docs/rules/restrict-template-expressions.md @@ -38,7 +38,7 @@ type Options = { }; const defaults = { - allowNumber: false, + allowNumber: true, allowBoolean: false, allowAny: false, allowNullable: false, diff --git a/packages/eslint-plugin/src/rules/restrict-template-expressions.ts b/packages/eslint-plugin/src/rules/restrict-template-expressions.ts index aeda73af521..dd030cf5bc0 100644 --- a/packages/eslint-plugin/src/rules/restrict-template-expressions.ts +++ b/packages/eslint-plugin/src/rules/restrict-template-expressions.ts @@ -41,7 +41,11 @@ export default util.createRule({ }, ], }, - defaultOptions: [{}], + defaultOptions: [ + { + allowNumber: true, + }, + ], create(context, [options]) { const service = util.getParserServices(context); const typeChecker = service.program.getTypeChecker(); diff --git a/packages/eslint-plugin/tests/rules/restrict-template-expressions.test.ts b/packages/eslint-plugin/tests/rules/restrict-template-expressions.test.ts index b0b3069062d..2d1a7422183 100644 --- a/packages/eslint-plugin/tests/rules/restrict-template-expressions.test.ts +++ b/packages/eslint-plugin/tests/rules/restrict-template-expressions.test.ts @@ -206,6 +206,7 @@ ruleTester.run('restrict-template-expressions', rule, { code: ` const msg = \`arg = \${123}\`; `, + options: [{ allowNumber: false }], errors: [ { messageId: 'invalidType', @@ -246,6 +247,7 @@ ruleTester.run('restrict-template-expressions', rule, { declare const arg: number; const msg = \`arg = \${arg}\`; `, + options: [{ allowNumber: false }], errors: [ { messageId: 'invalidType',