[code-infra] Replace all references of event as e
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 99.1k
- Forks
- 32.5k
- Avg merge
- 2d 17h
- Merged PRs (30d)
- 106
Description
Also enforce this with an eslint rule. e.g. https://eslint.org/docs/latest/rules/id-length https://eslint.org/docs/latest/rules/id-denylist
Single letter identifiers are desired in some cases (t for translations, and i, j, k for loop variables). So let's keep the scope of this issue down to specific identifiers only and not enforce it by length. Some other ids I can think of are:
err=>errorres=>response, or did you meanresult, orresolve?- ?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with docs/data/joy/components/list/ExampleNavigationMenu.tsx around the referenced event identifier, then search the repository for the specified short names. Review ESLint's id-denylist rule and define exceptions for intended identifiers such as t, i, j, and k. Done means the targeted references are expanded and the lint rule enforces those names consistently.
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
- Mostly clear
- Newbie friendliness
- 35/100