Khan / perseus / 5956285679

69%

push

github

Convert all JSX.LibraryManagedAttributes to use PropsFor<> (#691)

## Summary:

Previously in Flow we had a simple type for exatracting prop types of a component while reatining their optionality: `React.ElementConfig<typeof MyComponent>`. In TypeScript the equivalent is alot uglier:

```javascript 
JSX.LibraryManagedAttributes<
   typeof MyComponent,
   React.ComponentProps<typeof MyComponent>
>
```

Wonder Stuff Core recently added a helper named `PropsFor<>` that works like Flow's `ElmeentConfig<T>` and so this PR adapts to use that new type.

One related fix I made was to remove the `context` object from the `Renderer`'s constructor. That's a deprecated pattern in React and I couldn't find any usages that do that. 

Issue: "none"

## Test plan:

`yarn typecheck`
`yarn test` (just to be safe)

Author: jeremywiebe

Reviewers: somewhatabstract, #perseus

Required Reviewers:

Approved By: somewhatabstract

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

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

7627 of 16592 branches covered (45.97%)

Branch coverage included in aggregate %.

25 of 25 new or added lines in 9 files covered. (100.0%)

71407 of 97424 relevant lines covered (73.3%)

710.18 hits per line