Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Flow #102

Merged
merged 10 commits into from
Dec 18, 2017
Merged

Added Flow #102

merged 10 commits into from
Dec 18, 2017

Conversation

notatestuser
Copy link
Contributor

@notatestuser notatestuser commented Dec 11, 2017

Requested in #36.

Flow is a static type checker for JavaScript. More on Flow:

Type Inference
Using data flow analysis, Flow infers types and tracks data as it moves through your code. You don't need to fully annotate your code before Flow can start to find bugs.

JavaScript, Your Way
Flow is designed to understand idiomatic JavaScript. It understands common JavaScript patterns and many of the weird things we JavaScript developers love to do.

Realtime Feedback
Flow gives you fast feedback while you code by incrementally rechecking your code as you make changes.

Easy Integration
Flow integrates well with many tools, making it easy to insert into your existing workflow and toolchain.

Implementation notes:

No changes to the existing build scripts. Already caught a couple of issues in utils. It's entirely opt-in through adding a comment // @flow on the first line of a file and you can start with //@flow weak and fix up the errors reported by npm run lint until the flag can be removed to reveal the remaining errors.

There's a extension for VSCode that works pretty well and displays coverage as you code:

Flow in VSCode

This is only covering src/utils.js for now. More to come!

@snowypowers snowypowers merged commit 3ff3ba9 into CityOfZion:dev Dec 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants