Skip to content

Commit cd9fde2

Browse files
committed
feature: eslint-plugin-putout: safe: remove-useless-push: add
1 parent 7940e4c commit cd9fde2

File tree

4 files changed

+22
-0
lines changed

4 files changed

+22
-0
lines changed

packages/eslint-plugin-putout/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,7 @@ Disabled 🐊**Putout** rules:
153153
-[`for-of/remove-useless`](https://github.com/coderaiser/putout/tree/v29.7.1/packages/plugin-for-of#remove-useless);
154154
-[`for-of/remove-unused-variables`](https://github.com/coderaiser/putout/tree/29.7.1/packages/plugin-for-of#remove-unused-variables);
155155
-[`maybe/noop`](https://github.com/coderaiser/putout/tree/29.2.4/packages/plugin-maybe#noop);
156+
-[`remove-useless-push`](https://github.com/coderaiser/putout/tree/38.1.2/packages/plugin-remove-useless-push#readme);
156157

157158
### safe+align
158159

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,7 @@ const safeRules = {
132132
'for-of/remove-unused-variables': 'off',
133133
'for-of/remove-useless': 'off',
134134
'maybe/noop': 'off',
135+
'remove-useless-push': 'off',
135136
};
136137

137138
module.exports.safeRules = safeRules;
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
const list = [];
2+
3+
for (const {isDir, name} of files) {
4+
if (!isDir)
5+
list.push(name);
6+
}

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

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,3 +32,17 @@ test('eslint-plugin-putout: safe: no-merge-duplicate-functions', async ({noProce
3232
},
3333
});
3434
});
35+
36+
test('eslint-plugin-putout: safe: no-merge-duplicate-functions', async ({noProcess}) => {
37+
await noProcess('no-remove-useless-push', {
38+
extends: ['plugin:putout/safe'],
39+
rules: {
40+
'putout/putout': ['error', {
41+
ignore: ['!**/fixture'],
42+
rules: {
43+
...safeRules,
44+
},
45+
}],
46+
},
47+
});
48+
});

0 commit comments

Comments
 (0)