Skip to content

Commit e78d8bc

Browse files
committed
feature: eslint-plugin-putout: safe rules: convert-template-to-string
1 parent 85533c9 commit e78d8bc

File tree

8 files changed

+21
-5
lines changed

8 files changed

+21
-5
lines changed

packages/eslint-plugin-putout/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,7 @@ Disabled **ESLint** rules:
172172
Disabled 🐊**Putout** rules:
173173

174174
-[`apply-template-literals`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-remove-empty#readme);
175+
-[`convert-template-to-string`](https://github.com/coderaiser/putout/tree/v40.0.0/packages/convert-template-to-string#readme);
175176
-[`remove-empty`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-remove-empty#readme);
176177
-[`merge-duplicate-functions`](https://github.com/coderaiser/putout/tree/v37.12.0/packages/plugin-merge-duplicate-functions#readme);
177178
-[`nodejs/remove-process-exit`](https://github.com/coderaiser/putout/tree/v29.0.0/packages/plugin-nodejs#remove-process-exit#readme);

packages/eslint-plugin-putout/lib/index.mjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ export const recommended = [
7171

7272
export const safeRules = {
7373
'apply-template-literals': 'off',
74+
'convert-template-to-string': 'off',
7475
'remove-empty': 'off',
7576
'merge-duplicate-functions': 'off',
7677
'nodejs/remove-process-exit': 'off',
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
console.log(`${s}`);

packages/eslint-plugin-putout/test/safe/fixture/no-merge-duplicate-functions.js renamed to packages/eslint-plugin-putout/test/safe/fixture/merge-duplicate-functions.js

File renamed without changes.

packages/eslint-plugin-putout/test/safe/fixture/no-remove-useless-declaration-fix.js renamed to packages/eslint-plugin-putout/test/safe/fixture/remove-useless-declaration-fix.js

File renamed without changes.

packages/eslint-plugin-putout/test/safe/fixture/no-remove-useless-declaration.js renamed to packages/eslint-plugin-putout/test/safe/fixture/remove-useless-declaration.js

File renamed without changes.

packages/eslint-plugin-putout/test/safe/fixture/no-remove-useless-push.js renamed to packages/eslint-plugin-putout/test/safe/fixture/remove-useless-push.js

File renamed without changes.

packages/eslint-plugin-putout/test/safe/safe.mjs

Lines changed: 18 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import {safe, safeRules} from '../../lib/index.mjs';
44
const test = createTest(import.meta.url, safe);
55

66
test('eslint-plugin-putout: safe: no-remove-useless-declaration', async ({process}) => {
7-
await process('no-remove-useless-declaration', {
7+
await process('remove-useless-declaration', {
88
rules: {
99
'putout/putout': ['error', {
1010
ignore: ['!**/fixture'],
@@ -16,8 +16,8 @@ test('eslint-plugin-putout: safe: no-remove-useless-declaration', async ({proces
1616
});
1717
});
1818

19-
test('eslint-plugin-putout: safe: no-merge-duplicate-functions', async ({noProcess}) => {
20-
await noProcess('no-merge-duplicate-functions', {
19+
test('eslint-plugin-putout: safe: merge-duplicate-functions: off', async ({noProcess}) => {
20+
await noProcess('merge-duplicate-functions', {
2121
rules: {
2222
'putout/putout': ['error', {
2323
ignore: ['!**/fixture'],
@@ -29,8 +29,21 @@ test('eslint-plugin-putout: safe: no-merge-duplicate-functions', async ({noProce
2929
});
3030
});
3131

32-
test('eslint-plugin-putout: safe: no-remoe-useless-push', async ({noProcess}) => {
33-
await noProcess('no-remove-useless-push', {
32+
test('eslint-plugin-putout: safe: remove-useless-push: off', async ({noProcess}) => {
33+
await noProcess('remove-useless-push', {
34+
rules: {
35+
'putout/putout': ['error', {
36+
ignore: ['!**/fixture'],
37+
rules: {
38+
...safeRules,
39+
},
40+
}],
41+
},
42+
});
43+
});
44+
45+
test('eslint-plugin-putout: safe: convert-template-to-string: off', async ({noProcess}) => {
46+
await noProcess('convert-template-to-string', {
3447
rules: {
3548
'putout/putout': ['error', {
3649
ignore: ['!**/fixture'],

0 commit comments

Comments
 (0)