kefir icon indicating copy to clipboard operation
kefir copied to clipboard

Maintain TypeScript types?

Open mAAdhaTTah opened this issue 6 years ago • 4 comments

We maintain types for Flow, but there are TS types in DefinitelyTyped. I have an open PR to update them, but I'm wondering if maybe it makes more sense to pull them in and maintain them ourselves? It would be easier to then keep the Flow & TS types in sync.

mAAdhaTTah avatar Jan 22 '19 14:01 mAAdhaTTah

I'm in favor of pulling them in.

We can make our tests sanity check the definitions by adding a tsconfig.json that includes "noEmit": true, adding typescript to devDependencies, and adding && tsc to package.json's scripts.test.

Macil avatar Jan 22 '19 19:01 Macil

Any change of decision here or still thinking you'll pull the types in? Happy to help if I can

gamb avatar Oct 01 '20 11:10 gamb

@gamb I'm still open to it but I haven't had to make changes since the PR I opened in the OP so it hasn't been a high priority. If you want to PR in the types & handle deprecating them in the DT repo, I'd approve it.

mAAdhaTTah avatar Oct 01 '20 12:10 mAAdhaTTah

@mAAdhaTTah Thanks. I'll try to find some time to put something up 👍

gamb avatar Oct 01 '20 13:10 gamb