eslint-config-cleanjs icon indicating copy to clipboard operation
eslint-config-cleanjs copied to clipboard

Idea: Enforce 100% covering by Flow

Open steida opened this issue 7 years ago • 6 comments

Atom somehow shows how much code is covered by Flow. CleanScript demands 100% I guess.

steida avatar Sep 08 '16 23:09 steida

This is it https://github.com/gajus/eslint-plugin-flowtype

steida avatar Nov 04 '16 22:11 steida

@steida what do you mean by "this is it"? I can't see any rules in eslint-plugin-flowtype to enforce 100% flow coverage.

yantakus avatar Mar 24 '17 15:03 yantakus

@yantakus I mean, we can add such rule.

steida avatar Mar 24 '17 15:03 steida

Yes, would be great.

yantakus avatar Mar 24 '17 16:03 yantakus

I totally second this.

r01010010 avatar Jun 01 '17 13:06 r01010010

I also think Flow would be an amazing compliment to this project. I found this repo through my interest in elm, and think that for somebody initially interested in elm, that person would usually give preference to flow instead of typescript, because flow shares the goal of no runtime exceptions with elm. (These are some pretty good slides, and this one in particular really hit home for me: https://djcordhose.github.io/flow-vs-typescript/2016_hhjs.html#/21)

Flow gives you more type safety per type signature: https://medium.com/the-thinkmill/adopting-flow-typescript-3549a3a36d51

And you don't have potential tooling conflicts because of the different file extension.

On 01/06/2017 8:23 AM, "Paloma Jiménez" [email protected] wrote:

I totally second this.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/bodil/eslint-config-cleanjs/issues/11#issuecomment-305491576, or mute the thread https://github.com/notifications/unsubscribe-auth/AAg8qLG5L9k1YAvZE7NGIZ6icsHYqdQ-ks5r_rs7gaJpZM4J4i38 .

devinrhode2 avatar Jun 09 '17 02:06 devinrhode2