Convert to TypeScript

[Methuselah96]

This is an issue to track the progress of converting this project to TypeScript. The plan is to do this incrementally in order to make the transition as smooth as possible and not make the review process too overwhelming. The overall plan is:

1. Incrementally convert docs to TypeScript.

• TypeScript files can import Flow files if they have a corresponding TypeScript declaration file, so we'll just do a breadth-first conversion.
• Import react-select types from @types/react-select in this step.
• Fix types in @types/react-select to make it work with docs.
• PRs:
  • Add TypeScript infrastructure (Add TypeScript infrastructure #4139)
  • Convert Webpack config to TypeScript (Convert Webpack config to TypeScript #4143)
  • Incrementally convert docs to TypeScript (part 1) (Incrementally convert docs to TypeScript (part 1) #4144)
  • Incrementally convert docs to TypeScript (part 2) (TODO)
  • etc.

2. Move Typescript declaration files from @types/react-select to this repository.
3. Incrementally convert main source files to TypeScript.

[bladey]

Thank you for all your efforts @Methuselah96, this looks great.

[Methuselah96]

@kylemh Thanks for all your help so far on PRs! I saw on #4293 that you're looking to help out with the TS conversion. Most of the work is actually already done. (I have the entire code-base converted to TypeScript on a branch, it's just a matter of figuring out how best to merge it in.)

The PRs I've made so far have been to incrementally convert the docs to TypeScript using the types that currently exist on DefinitelyTyped. When I run across a problem in the types, I make a PR to DefinitelyTyped. This works, however it's slow because there are no active reviewers for react-select on DefinitelyTyped. Would be willing to update the types on DefinitelyTyped to 3.1 (by adding the isLoading prop to AsyncSelect) and also add yourself as a definition author? That way we can review each others PRs on DefinitelyTyped and this process can be sped up quite a bit.

Thanks for all your hard work!

[kylemh]

Down; however, that looks to already be done, no?

[Methuselah96]

I think it's in the State, but not in the Props.

[kylemh]

Async extends State. Async represents the inferred props on the default export of /async:

[kylemh]

I did reference the wrong line of code though. The AsyncSelect props extends the "NamedProps" interface -> https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-select/src/Select.d.ts#L215 which has isLoading

[Methuselah96]

I see now, the PR that added the isLoading prop to Async just made it so that it was passed through the Select properly. My bad. Can you just make a PR to bump the version number to 3.1 and add yourself as an author? Or you can find an actual change to make; there are quite a few of missing/incorrect types (for example, tightening up these styles types so they don't pass any as the props).

[kylemh]

Certainly! My work week is a bit busy, but I'll come back to this at some point next week at the worst.

[Methuselah96]

No worries, I can relate. Just whenever you find the time.