danger / JavaScript

dangerJS does't understand `// @noflow`?

dangerJS does't understand `// @noflow`?

danger/JavaScript · June 2, 2018 at 10:28pm

dangerJS does't understand `// @noflow`?

danger / JavaScript · June 2, 2018 at 10:28pm
i'm facing following warn, and i push commit with added `// @noflow`, but warn still remain on PR.
These modified files do not have Flow enable

`// @noflow` is not suppert now?

this is the PR Link:
https://github.com/withspectrum/spectrum/pull/3236#issuecomment-394111224

thank you for any feedbacks!
ajust desktop draggable region area by ryota-murakami · Pull Request…
https://github.com/withspectrum/spectrum/pull/3236

June 3, 2018 at 12:38pm

That's not danger directly, but the flow plugin i wrote: https://github.com/withspectrum/danger-plugin-flow

  • reply
  • like

PRs adding support for @noflow directives much appreciated!

  • reply
  • like

oh, thanks comments and this is awesome work!

i'd like to sent @noflow PR!

  • reply
  • like
Your message here...

*bold*_italic_`code````codeblock```