Khan / perseus / 5579394208

69%

push

github

Bump flowgen to handle forwardRef components better (#614)

## Summary:

I was noticing that our Flow types were causing a syntax error. Tracked it down to incorrect Flow type generation for forward ref components. Bumping to a newer version of `flowgen` fixes this.

Issue: LC-950

## Test plan:

`yarn; yarn build:flowtypes`

Review the generated types for `ServerItemRenderer` (`packages/perseus/dist/server-item-renderer.js.flow`) and verify it doesn't use types that don't exist in the React Flow libdefs.

Author: jeremywiebe

Reviewers: kevinbarabash, #perseus

Required Reviewers:

Approved By: kevinbarabash

Checks: ✅ finish_coverage, ✅ Publish npm snapshot (ubuntu-latest, 16.x), ✅ Check builds for changes in size (ubuntu-latest, 16.x), ✅ Lint, Typecheck, and Test (ubuntu-latest, 16.x), ✅ Extract i18n strings (ubuntu-latest, 16.x), ✅ Cypress Coverage (ubuntu-latest, 16.x), ✅ Jest Coverage (ubuntu-latest, 16.x), ✅ Check for .changeset file (ubuntu-latest, 16.x), ✅ gerald

Pull Request URL: https://github.com/Khan/perseus/pull/614

7559 of 16353 branches covered (46.22%)

Branch coverage included in aggregate %.

69764 of 95681 relevant lines covered (72.91%)

718.99 hits per line