import-js/eslint-plugin-import

no-unused-modules doesn't report unused Flow type exports

Open

#1,540 建立於 2019年11月13日

在 GitHub 查看
 (0 留言) (0 反應) (0 負責人)JavaScript (4,946 star) (1,540 fork)batch import
bugflowhelp wanted

描述

// foo.js

// @flow strict
export type Foo = string
export default ''

// index.js

// @flow strict
import foo from './foo'

Expected: export type Foo is reported as unused Actual: no errors

貢獻者指南

技術棧
javascript
領域
tooling
議題類型
bug
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 hours
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
stale
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
knowledge of ESLintunderstanding of Flow typesfamiliarity with the no unused modules rule
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
35
研究方向
Investigate the `no unused modules` rule implementation in `src/rules/no unused modules.js`. Focus on how type exports are processed—Flow type exports may not be recognized as exports in the module graph. Check the parsing logic for `// @flow` annotations and ensure that Flow type exports are included in the list of exports. Look at existing tests for patterns and consider adding a test case for Flow type exports.