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