browserify / resolve

Implements the node.js require.resolve() algorithm
MIT License
775 stars 183 forks source link

Added @noflow in malformed package #282

Closed sachp1 closed 2 years ago

sachp1 commented 2 years ago

Added @noflow to bypass flow check for test file

flow server shows the malformed package as an error.

adding noflow tag will allow flow to ignore this file.

ljharb commented 2 years ago

Duplicate of #270. Duplicate of #269. See #262.

This is a test file. It is supposed to be malformed.

Flow should not be checking files in node_modules anyways, but it certainly should not be checking JSON files that aren't imported, and even more so, it should not be failing when files not under your control have invalid syntax. This is a Flow bug.

sachp1 commented 2 years ago

issue was solved after ignoring node modules in .flowconfig