[Proposal] Enabling `no-param-reassign` eslint rule
未关闭
还没有人认领这个 Issue。
never-stale
- 主要语言
- JavaScript
- 星标
- 122k
- 派生
- 37.4k
- 平均合并
- 4 天 3 小时
- 30 天内合并 PR
- 272
描述
I've personally often feel frustrated due to being extremely hard to track an implementation after a reassignment of parameters, due to the ambiguity of types in a function that has a JSDoc declaration.
I recommend enabling https://eslint.org/docs/latest/rules/no-param-reassign. There is currently ✖ 1597 problems (1597 errors, 0 warnings).
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
首先检查提议的 ESLint 规则 no-param-reassign,并运行仓库现有的 ESLint 检查。当前的 lint 运行报告了 1,597 个错误;完成的标准是启用该规则,并解决这些参数重新赋值违规,同时不引入新的 lint 错误。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- javascript
- 领域
- tooling
- Issue 类型
- 重构
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 冷清
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100