facebook / facebook/flow

Should [ignore] apply to name_mapper aliases?

未關閉
#2,973 0 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
主要語言
Rust
星號
22.3k
分支
1.9k
PR 合併指標
30 天內沒有已合併 PR

描述

`[ignore]` options affect `name_mapper` aliases too. This seems unusual. For example, I would expect this configuration to allow Flow to check and pass `require('./includes/style.scss')` with no errors.

```
[ignore]
.*/node_modules/.*

[options]
module.name_mapper.extension='scss' -> 'empty/object'
```

`empty` is [the npm package](https://www.npmjs.com/package/empty), installed by `npm install` into `node_modules/`.

This configuration fails with;

```
src/main.js:2
2: require('./includes/style.scss');
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ./includes/style.scss. Required module not found
```

If the ignore rule is removed it passes, but `flow check --all` is slow and fails on the content of `node_modules/`.

E.g https://github.com/BevanR/Flow-type-ignores-aliases-too

How can I include the `empty` npm package for the purpose of name_mapping/aliasing without also type-checking `/node_modules/` or resorting to `// @flow` comments in all of my files (instead of `flow check --all`)?

Is this supported?

貢獻指南

開啟貢獻指南

評估

這個 Issue 還沒有評估資料。

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。