Commit d324207
authored
chore: Temporarily disable
## Explanation
The rule `@typescript-eslint/naming-convention` was already disabled for
most files, but it was set to `warn` in a few places still. It's now
completely disabled instead.
This was done to reduce the set of changes that are part of the upcoming
switch to error suppression (#6790), and also for consistency. We still
want to use this rule, but we can gradually begin enforcing it in a more
controlled manner using error suppression, across the entire repository
at once rather than selectively.
## References
Relates to #6790
These changes were extracted from the draft PR #7148
## Checklist
- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've communicated my changes to consumers by [updating changelogs
for packages I've
changed](https://github.com/MetaMask/core/tree/main/docs/contributing.md#updating-changelogs),
highlighting breaking changes as necessary
- [x] I've prepared draft pull requests for clients and consumer
packages to resolve any breaking changes
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> Disables `@typescript-eslint/naming-convention` everywhere and removes
related thresholds/config and an inline disable comment.
>
> - **ESLint config**
> - Remove `@typescript-eslint/naming-convention: 'warn'` from
`**/*.d.ts` and `packages/eth-block-tracker/**/*.ts` overrides in
`eslint.config.mjs`.
> - **Warning thresholds**
> - Delete thresholds for
`packages/eth-block-tracker/tests/buildDeferred.ts` and remove
`@typescript-eslint/naming-convention` entry from
`tests/setupAfterEnv.ts` in `eslint-warning-thresholds.json`.
> - **Types**
> - Drop inline disable for `@typescript-eslint/naming-convention` in
`types/global.d.ts` comment.
>
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
4bc7432. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->@typescript-eslint/naming-convention (#7151)1 parent 49615f6 commit d324207
File tree
3 files changed
+1
-7
lines changed- types
3 files changed
+1
-7
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
101 | 101 | | |
102 | 102 | | |
103 | 103 | | |
104 | | - | |
105 | | - | |
106 | | - | |
107 | 104 | | |
108 | 105 | | |
109 | 106 | | |
110 | 107 | | |
111 | 108 | | |
112 | | - | |
113 | 109 | | |
114 | 110 | | |
115 | 111 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
188 | 188 | | |
189 | 189 | | |
190 | 190 | | |
191 | | - | |
192 | 191 | | |
193 | 192 | | |
194 | 193 | | |
| |||
224 | 223 | | |
225 | 224 | | |
226 | 225 | | |
227 | | - | |
228 | 226 | | |
229 | 227 | | |
230 | 228 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
| 8 | + | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| |||
0 commit comments