Skip to content

Commit 2542a53

Browse files
committed
feature: @putout/plugin-eslint-plugin: apply-is-space-between: add
1 parent 77c127b commit 2542a53

File tree

9 files changed

+56
-0
lines changed

9 files changed

+56
-0
lines changed

packages/plugin-eslint-plugin/README.md

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ npm i @putout/plugin-eslint-plugin -D
2020
-[apply-flat-config-to-rule-tester](#apply-flat-config-to-rule-tester);
2121
-[apply-get-token-before](#apply-get-token-before);
2222
-[apply-get-token-after](#apply-get-token-after);
23+
-[apply-is-space-between](#apply-is-space-between);
2324
-[convert-context-to-source](#convert-context-to-source);
2425
-[convert-require-resolve-to-require](#convert-require-resolve-to-require);
2526
-[turn-off-schema](#turn-off-schema);
@@ -34,6 +35,7 @@ npm i @putout/plugin-eslint-plugin -D
3435
"eslint-plugin/apply-flat-config-to-rule-tester": "on",
3536
"eslint-plugin/apply-get-token-before": "on",
3637
"eslint-plugin/apply-get-token-after": "on",
38+
"eslint-plugin/apply-is-space-between": "on",
3739
"eslint-plugin/convert-context-to-source": "on",
3840
"eslint-plugin/convert-require-resolve-to-require": "on",
3941
"eslint-plugin/turn-off-schema": "on",
@@ -131,6 +133,26 @@ source.getTokenAfter(token, {
131133
});
132134
```
133135

136+
## apply-is-space-between
137+
138+
> The following deprecated SourceCode methods have been removed in ESLint v10.0.0: `isSpaceBetweenTokens()`.
139+
>
140+
> (c) [eslint.org](https://eslint.org/docs/next/use/migrate-to-10.0.0#sourcecode-methods-removed)
141+
142+
Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/025255fed985b168a60c1ea7f37ec3fb/31e8e077c1a175efc2f8d3f3f4cb8e2e88ef7192).
143+
144+
### ❌ Example of incorrect code
145+
146+
```js
147+
isSpaceBetweenTokens(first, second);
148+
```
149+
150+
### ✅ Example of correct code
151+
152+
```js
153+
isSpaceBetween(first, second);
154+
```
155+
134156
## convert-context-to-source
135157

136158
> When **ESLint** v9.0.0 is released, it will ship with several breaking changes for rule authors.
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
isSpaceBetween(first, second);
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
isSpaceBetweenTokens(first, second)
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
export const report = () => `Use 'isSpaceBetween' instead of 'isSpaceBetweenTokens'`;
2+
3+
export const replace = () => ({
4+
'isSpaceBetweenTokens(__a, __b)': 'isSpaceBetween(__a, __b)',
5+
});
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
import {createTest} from '@putout/test';
2+
import * as plugin from './index.js';
3+
4+
const test = createTest(import.meta.url, {
5+
plugins: [
6+
['apply-is-space-between', plugin],
7+
],
8+
});
9+
10+
test('eslint-plugin: apply-is-space-between: report', (t) => {
11+
t.report('apply-is-space-between', `Use 'isSpaceBetween' instead of 'isSpaceBetweenTokens'`);
12+
t.end();
13+
});
14+
15+
test('eslint-plugin: apply-is-space-between: transform', (t) => {
16+
t.transform('apply-is-space-between');
17+
t.end();
18+
});

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import * as applyIsSpaceBetween from './apply-is-space-between/index.js';
12
import * as applyGetTokenAfter from './apply-get-token-after/index.js';
23
import * as applyGetTokenBefore from './apply-get-token-before/index.js';
34
import * as removeErrorsType from './remove-errors-type/index.js';
@@ -16,4 +17,5 @@ export const rules = {
1617
'remove-errors-type': removeErrorsType,
1718
'apply-get-token-before': applyGetTokenBefore,
1819
'apply-get-token-after': applyGetTokenAfter,
20+
'apply-is-space-between': applyIsSpaceBetween,
1921
};

packages/plugin-eslint-plugin/test/eslint-plugin.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,3 +56,8 @@ test('plugin-eslint-plugin: transform: apply-get-token-after', (t) => {
5656
t.transform('apply-get-token-after');
5757
t.end();
5858
});
59+
60+
test('plugin-eslint-plugin: transform: apply-is-space-between', (t) => {
61+
t.transform('apply-is-space-between');
62+
t.end();
63+
});
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
isSpaceBetween(first, second);
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
isSpaceBetweenTokens(first, second)

0 commit comments

Comments
 (0)