sindresorhus/eslint-plugin-unicorn

Improve `prefer-dom-node-remove`

已关闭

#2,085 创建于 2023年5月9日

在 GitHub 查看贡献者指南
 (0 条评论) (0 个反应) (0 位负责人)JavaScript (468 个派生)user submission
enhancementhelp wanted

仓库指标

星标
 (5,022 个星标)
PR 合并指标
 (平均合并 1天 16小时) (30 天内合并 399 个 PR)

描述

  • Use snapshot test https://github.com/sindresorhus/eslint-plugin-unicorn/pull/2333
  • Improve fix, not use the original text but manipulate tokens
  • Improve fix for node?.parentNode.removeChild(node) (node can be anything, the object and the argument should be same reference, but can be different AST, eg: a?.b?.parentNode.remove(a.b))

贡献者指南

研究方向
检查 `rules/prefer dom node remove.js` 中现有的修复器（可能在仓库中）。查看已合并的 PR #2333 以了解快照测试风格。对于第二个任务，通过访问 `sourceCode.getTokens()` 或类似方法获取 AST 令牌，以替换原始文本的令牌操作。对于第三个任务，处理可选链（例如 `node?.parentNode.removeChild(node)`），确保在对象和参数中使用相同的引用；考虑使用 `sourceCode.getText()` 或比较 AST 节点。参考 `test/snapshots/prefer dom node remove.md` 中的现有测试以获取指导。
技术栈
javascriptnodejs
领域
developer experiencetooling
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
Experience with ESLint rulesUnderstanding of ASTFamiliarity with fixer API
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
50