import-js/eslint-plugin-import

Disallow duplicate require

Open

#2,925 opened on 2023年11月18日

GitHub で見る
 (9 comments) (1 reaction) (0 assignees)JavaScript (4,946 stars) (1,540 forks)batch import
enhancementhelp wanted

説明

I have following code :

const { addCellToRow } = require("./cell-utils.js");
const { setCellValue, findCell, createCell } = require("./cell-utils.js");

Both require are using the same path, and should thus be grouped.

I've tried to enable this in my .eslintrc :

"import/no-duplicates": ["error", {"prefer-inline": true}],

But it didn't have any effect.

Are there any rules that can catch this ?

コントリビューターガイド

技術スタック
javascriptnodejs
領域
tooling
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
active
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Understanding of ESLint plugin developmentFamiliarity with AST for require statements
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
Investigate the existing `import/no duplicates` rule to understand why it does not catch duplicate `require()` calls. Check the rule's source code in the repository to see how it handles `import` vs `require`. Look at the issue comments for any suggested fixes or workarounds. Consider whether a new rule or an enhancement to the existing rule is needed.
Disallow duplicate require · import-js/eslint-plugin-import#2925 | Good First Issue