Flow PR review Notes

[stevenjmarsh]

Items to review:

• quick look over configuration (.flowconfig, .babelrc), make sure there aren't any obvious mistakes, or inefficiencies (all does seem to be working at this point)
• Look at react-toggle libdef in .types/
  • works partially
    • it does correctly handle / suppress Required module not found...
    • it does correctly handle the expected export type (a React component)
    • it does not seem to enforce prop type checking (i.e., set id to number, still succeeds)
  • the libdef (declare) is in file react-toggle.js, and Toggle component is imported / used in Filter.jsx
  • I followed some examples in flow type definitions, like this
• Look at type aliases (definitions) and type annotations in Filter.jsx
  • confirm they are effective and suggest any convention changes

[stevenjmarsh]

I moved these notes that I was capturing, to the pull request comments.