import-js/eslint-plugin-import

no-unused-modules "missingExports" returns false positives

Open

#2,102 opened on 2021年5月27日

GitHub で見る
 (7 comments) (1 reaction) (0 assignees)JavaScript (4,946 stars) (1,540 forks)batch import
`exports` supportbughelp wanted

説明

I'd expect both of these cases to count as exports:

module.exports = {} // Should probably not trigger

export * from './someFile' // Should almost definitely not trigger

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

技術スタック
javascriptnodejs
領域
toolingdeveloper experience
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
eslintnpm module exports
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
The bug is in the no unused modules rule's missingExports logic. Examine the rule's source at src/rules/no unused modules.js and its tests in tests/src/rules/no unused modules.js. The issue involves detecting exports from `module.exports = {}` and `export * from './someFile'`. Look at how the rule currently gathers exports and why these cases are missed. Check GitHub for any existing PRs or comments that discuss the detection scope. The fix likely requires modifying the export extraction to include these patterns, ensuring they are considered as exports.
no-unused-modules "missingExports" returns false positives · import-js/eslint-plugin-import#2102 | Good First Issue