eslint-plugin-putout 10.10.0 → 11.3.0

package/README.md

@@ -49,6 +49,7 @@ Then configure the rules you want to use under the rules section.
         "putout/newline-function-call-arguments": "error",
         "putout/function-declaration-paren-newline": "error",
         "putout/remove-newline-after-default-import": "error",
+        "putout/remove-empty-newline-after-last-specifier": "error",
         "putout/objects-braces-inside-array": "error",
         "putout/object-init": "error"
     }
@@ -68,11 +69,13 @@ Then configure the rules you want to use under the rules section.
 - [Newline function call arguments](/packages/eslint-plugin-putout/lib/newline-function-call-arguments)
 - [Function declaration paren newline](/packages/eslint-plugin-putout/lib/function-declaration-paren-newline)
 - [Remove newline after default import](/packages/eslint-plugin-putout/lib/remove-newline-after-default-import)
+- [Remove empty newline after last specifier](/packages/eslint-plugin-putout/lib/remove-empty-newline-after-last-specifier)
 - [Objects braces inside array](/packages/eslint-plugin-putout/lib/objects-braces-inside-array)
 - [Object init](/packages/eslint-plugin-putout/lib/object-init)
 - [No unresolved](/packages/eslint-plugin-putout/lib/no-unresolved)
 - [Evaluate](/packages/eslint-plugin-putout/lib/evaluate)
 - [Tape: add new line before assertion]('/packages/eslint-plugin-putout/lib/tape-add-new-line-before-assertion)
+- [Tape: add new line between tests]('/packages/eslint-plugin-putout/lib/tape-add-new-line-between-tests)
 
 ### Safe mode
 
@@ -96,6 +99,7 @@ List of disabled `putout` rules:
 - [remove-unused-types](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-unused-types);
 - [remove-unused-for-of-variables](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-unused-for-of-variables);
 - [remove-unused-expressions](https://github.com/coderaiser/putout/tree/v20.0.0/packages);
+- [remove-useless-return](https://github.com/coderaiser/putout/tree/master/remove-useless-return);
 - [remove-skip](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-skip);
 - [remove-only](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-only);
 - [remove-console](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-console);

package/lib/index.js

@@ -26,11 +26,13 @@ module.exports.rules = {
     ...getWrapRule('newline-function-call-arguments'),
     ...getWrapRule('function-declaration-paren-newline'),
     ...getWrapRule('remove-newline-after-default-import'),
+    ...getWrapRule('remove-empty-newline-after-last-specifier'),
     ...getWrapRule('objects-braces-inside-array'),
     ...getWrapRule('object-init'),
     ...getWrapRule('no-unresolved'),
     ...getWrapRule('evaluate'),
     ...getWrapRule('tape-add-newline-before-assertion'),
+    ...getWrapRule('tape-add-newline-between-tests'),
     ...getRule('putout'),
 };
 
@@ -54,11 +56,13 @@ const recommended = {
         'putout/newline-function-call-arguments': 'error',
         'putout/function-declaration-paren-newline': 'error',
         'putout/remove-newline-after-default-import': 'error',
+        'putout/remove-empty-newline-after-last-specifier': 'error',
         'putout/objects-braces-inside-array': 'error',
         'putout/object-init': 'error',
         'putout/no-unresolved': 'error',
         'putout/evaluate': 'error',
         'putout/tape-add-newline-before-assertion': 'error',
+        'putout/tape-add-newline-between-tests': 'error',
         'putout/putout': 'error',
         
         'node/no-unsupported-features/es-syntax': 'off',
@@ -87,8 +91,9 @@ const safe = {
                 'remove-unused-variables': 'off',
                 'remove-unused-expressions': 'off',
                 'remove-unused-for-of-variables': 'off',
-                'remove-skip': 'off',
-                'remove-only': 'off',
+                'remove-useless-return': 'off',
+                'tape/remove-skip': 'off',
+                'tape/remove-only': 'off',
                 'remove-console': 'off',
                 'remove-debugger': 'off',
                 'convert-for-to-for-of': 'off',

package/lib/remove-empty-newline-after-last-specifier/README.md

@@ -0,0 +1,24 @@
+# Remove empty new line after last specifier(remove-empty-newline-after-last-specifier)
+
+## Rule Details
+
+This rule aims to remove empty newline after last specifier.
+
+Examples of **incorrect** code for this rule:
+
+```js
+import {
+    a,
+    b,
+
+} from 'y';
+```
+
+Examples of **correct** code for this rule:
+
+```js
+import {
+    a,
+    b,
+} from 'y';
+```

package/lib/remove-empty-newline-after-last-specifier/index.js

@@ -0,0 +1,20 @@
+'use strict';
+
+module.exports.category = 'import';
+module.exports.report = () => 'Remove newline after last specifier';
+
+const regExp = /\n\s+}/;
+
+module.exports.filter = ({text}) => {
+    return regExp.test(text);
+};
+
+module.exports.fix = ({text}) => {
+    return text
+        .replace(regExp, '\n}');
+};
+
+module.exports.include = () => [
+    'ImportDeclaration',
+];
+

package/lib/tape-add-newline-between-tests/README.md

@@ -0,0 +1,42 @@
+# Add new line between tests (add-newline-between-tests)
+
+Add new line between tests, for [supertape](https://github.com/coderaiser/supertape).
+
+## Rule Details
+
+This rule aims to add new line between tests.
+
+Examples of **incorrect** code for this rule:
+
+```js
+test('lint: do some check', (t) => {
+    const result = 1 + 2;
+    
+    t.equal(result, 3);
+    t.end();
+});
+test('lint: do some check', (t) => {
+    const result = 1 + 2;
+    
+    t.equal(result, 3);
+    t.end();
+});
+```
+
+Examples of **correct** code for this rule:
+
+```js
+test('lint: do some check', (t) => {
+    const result = 1 + 2;
+    
+    t.equal(result, 3);
+    t.end();
+});
+
+test('lint: do some check', (t) => {
+    const result = 1 + 2;
+    
+    t.equal(result, 3);
+    t.end();
+});
+```

package/lib/tape-add-newline-between-tests/index.js

@@ -0,0 +1,27 @@
+'use strict';
+
+module.exports.category = 'tape';
+module.exports.report = () => 'Add new line between tests';
+
+module.exports.filter = ({text, node, getText, getCommentsBefore}) => {
+    if (!/^test(\.only|\.skip)?\(/.test(text))
+        return false;
+    
+    const comments = getCommentsBefore(node);
+    
+    if (comments.length)
+        return false;
+    
+    const [a] = getText(node, 2);
+    
+    return a === ';';
+};
+
+module.exports.fix = ({text}) => {
+    return `\n${text}`;
+};
+
+module.exports.include = () => [
+    'CallExpression ',
+];
+

package/lib/wrap.js

@@ -6,6 +6,7 @@ const prepare = (plugin, context, options) => (node) => {
     const source = context.getSourceCode();
     const filename = context.getFilename();
     const getText = source.getText.bind(source);
+    const getCommentsBefore = source.getCommentsBefore.bind(source);
     
     const text = getText(node);
     
@@ -14,6 +15,7 @@ const prepare = (plugin, context, options) => (node) => {
         node,
         options,
         getText,
+        getCommentsBefore,
         filename,
     });
     

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "10.10.0",
+  "version": "11.3.0",
   "description": "eslint plugin for putout",
   "release": false,
   "tag": false,
@@ -58,8 +58,8 @@
     "node": ">=14"
   },
   "peerDependencies": {
-    "eslint": ">=8.0.0-beta.0",
-    "putout": ">=20"
+    "eslint": ">=8.0.0",
+    "putout": ">=21"
   },
   "license": "MIT",
   "publishConfig": {