facebook / facebook/flow

Should [ignore] apply to name_mapper aliases?

Đang mở
#2,973 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Rust
Star
22.3k
Fork
1.9k
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

`[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?

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.