suggestion: Add no-shadow to default set of rules
Open
- Dominant language
- TypeScript
- Stars
- 5.3k
- Forks
- 212
- PR merge metrics
- No merged PRs in 30d
Description
The [no-shadow](https://eslint.org/docs/latest/rules/no-shadow) is like a good default rule to have. I can think of many reasons why this would be a good default. Allowing variable shadowing can:
* Complicate scope resolution
* Make it difficult to understand code intent
* Complicates refactoring
And many others.
Contributor guide
Assessment
This issue has not been assessed yet.