Add TypeScript definition

[colinking]

This PR adds TS types to ink-link.

I based the typings on ink-text-link: https://github.com/vadimdemedes/ink-text-input

It also includes the fallback prop added in this PR: #5

[sindresorhus]

Thanks for creating a TypeScript definition. 🙌 Can you follow this styleguide?

Some examples:

• sindresorhus/semver-regex@039944b
• sindresorhus/global-directory@0aae4d6
• sindresorhus/md5-hex@bbbfeff

[sindresorhus]

Friendly bump :)

[colinking]

Hey @sindresorhus! Finally got around to updating this to use the styleguide.

One thing: I'm not sure which React type to use on an instance of a React Component -- do you?

[sindresorhus]

One thing: I'm not sure which React type to use on an instance of a React Component -- do you?

I don't. Maybe look at some other React package with a index.d.ts file and see how they do it?

[colinking]

I poked around on GitHub and found a number of folks using React + tsd, but none of them were checking the type of a React node: https://github.com/search?q=expectType+tsd+react&type=Code

From inspecting the createElement API, I tried expectType<FunctionComponentElement<Props>>(<Link/>) with no luck:

  index.test-d.tsx:6:0
  ✖  6:0  Parameter type FunctionComponentElement<Props> is not identical to argument type Element.
  ✖  7:0  Parameter type FunctionComponentElement<Props> is not identical to argument type Element.
  ✖  8:0  Parameter type FunctionComponentElement<Props> is not identical to argument type Element.

It seems like tsd is requesting we specifically type check against the React.Element type, which is not exported.

Give that, I just removed the expectType checks and replaced them with a simple function, since the type checks don't add a lot of value -- we already know it's valid JSX. The important thing here imo is to validate that the component properties are correct which tsd will do by compiling the declarations.

Does that seem good to you? :)

[sindresorhus]

Does that seem good to you? :)

👍