-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Closed
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issuerepo maintenancethings to do with maintenance of the repo, and not with code/docsthings to do with maintenance of the repo, and not with code/docs
Description
Suggestion
#11628 deprecated the type field, but didn't remove usages of type.
For example, this file has tons of them:
typescript-eslint/packages/eslint-plugin/tests/rules/no-shadow/no-shadow-eslint.test.ts
Line 30 in 3f02687
| type: AST_NODE_TYPES.Identifier, |
I had thought in #11582 that no-deprecated would enforce this, but it turns out it's a case of #10643 / microsoft/TypeScript#60442. So this issue is to go ahead and remove the usage in our tests.
Additional Info
JoshuaKGoldberg
Metadata
Metadata
Assignees
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issuerepo maintenancethings to do with maintenance of the repo, and not with code/docsthings to do with maintenance of the repo, and not with code/docs