MetaMask / MetaMask/snaps

Investigate disabled rules after ESLint 9 migration

Open
#3,132 0 comments 0 reactions 0 assignees View on GitHub
snaps-refined-backlog
Dominant language
TypeScript
Stars
853
Forks
662
Avg merge
2d 8h
Merged PRs (30d)
8

Description

Some lint violations were disabled after the migration. In those cases I added a comment "Either fix this lint violation or explain why it's necessary to ignore.". We should follow up on these and investigate further if we can fix these somehow.

Contributor guide

Open the contributing guide

Research direction

Start by locating the lint-violation comments added during the ESLint 9 migration and review the disabled rules around them. For each case, determine whether the violation can be fixed; done means the violation is fixed or the reason for keeping it disabled is documented.

Written by the indexing model from the issue text.

Assessment

Tech stack
eslint, typescript
Domain
tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.