Skip to content

Commit 20f23ad

Browse files
skovyMichaelDeBoey
authored andcommitted
feat(prefer-wait-for): remove rule (#648)
BREAKING CHANGE: `prefer-wait-for` is now removed
1 parent 6de1d96 commit 20f23ad

File tree

6 files changed

+2
-2549
lines changed

6 files changed

+2
-2549
lines changed

.github/ISSUE_TEMPLATE/propose_new_rule.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ body:
77
attributes:
88
label: Name for new rule
99
description: Suggest a name for the new rule that follows the [rule naming conventions](https://github.com/testing-library/eslint-plugin-testing-library/blob/main/CONTRIBUTING.md#rule-naming-conventions).
10-
placeholder: prefer-wait-for
10+
placeholder: prefer-find-by
1111
validations:
1212
required: true
1313

README.md

-1
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,6 @@ To enable this configuration use the `extends` property in your
231231
| [`prefer-query-by-disappearance`](./docs/rules/prefer-query-by-disappearance.md) | Suggest using `queryBy*` queries when waiting for disappearance | | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] ![marko-badge][] |
232232
| [`prefer-screen-queries`](./docs/rules/prefer-screen-queries.md) | Suggest using `screen` while querying | | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] ![marko-badge][] |
233233
| [`prefer-user-event`](./docs/rules/prefer-user-event.md) | Suggest using `userEvent` over `fireEvent` for simulating user interactions | | |
234-
| [`prefer-wait-for`](./docs/rules/prefer-wait-for.md) | Use `waitFor` instead of deprecated wait methods | 🔧 | |
235234
| [`render-result-naming-convention`](./docs/rules/render-result-naming-convention.md) | Enforce a valid naming for return value from `render` | | ![angular-badge][] ![react-badge][] ![vue-badge][] ![marko-badge][] |
236235

237236
<!-- RULES-LIST:END -->

docs/rules/prefer-wait-for.md

-74
This file was deleted.

lib/rules/prefer-wait-for.ts

-214
This file was deleted.

tests/index.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import plugin from '../lib';
88
const execAsync = util.promisify(exec);
99
const generateConfigs = () => execAsync(`npm run generate:configs`);
1010

11-
const numberOfRules = 27;
11+
const numberOfRules = 26;
1212
const ruleNames = Object.keys(plugin.rules);
1313

1414
// eslint-disable-next-line jest/expect-expect

0 commit comments

Comments
 (0)