react-sticky-table icon indicating copy to clipboard operation
react-sticky-table copied to clipboard

migrate to Typescript

Open horacioh opened this issue 5 years ago • 6 comments

is there any interest in migrating this package to Typescript?

I can help with the migration, I'm using it for a client in a Typescript codebase and I think it would be useful since the package is not that big.

let me know!

horacioh avatar Jun 05 '19 13:06 horacioh

Possibly!

It would be pretty quick as the 3.0.1 codebase is very small.

What would be involved besides changes to code? How would it affect existing non-ts projects?

Max

On Wed, Jun 5, 2019 at 6:09 AM Horacio Herrera [email protected] wrote:

is there any interest in migrating this package to Typescript?

I can help with the migration, I'm using it for a client in a Typescript codebase and I think it would be useful since the package is not that big.

let me know!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/henrybuilt/react-sticky-table/issues/61?email_source=notifications&email_token=AB4MRJ7UARDR3IN7CHQ4FPLPY63INA5CNFSM4HTXQO32YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GXYPM5Q, or mute the thread https://github.com/notifications/unsubscribe-auth/AB4MRJ67WY6DCBAC5YXECC3PY63INANCNFSM4HTXQO3Q .

maxhudson avatar Jun 05 '19 13:06 maxhudson

non-ts codebases will not be affected. maybe they will still benefit from it if you are using an IDE like VSCode that will still read the types for the components you use to give you some instrospection and help. :)

I can start doing the migration if you want

horacioh avatar Jun 05 '19 13:06 horacioh

What work would be involved besides changes to code? Would the package need to change its structure? Would we need to publish an additional file?

maxhudson avatar Jun 05 '19 14:06 maxhudson

we need to setup the project to work with typescript + publish the type definitions.. I can do all the setup in my fork and create a PR if you want!

horacioh avatar Jun 05 '19 19:06 horacioh

Yeah sounds good! Will be happy to accept if it doesn't create any issues for existing users. Thanks.

maxhudson avatar Jun 05 '19 21:06 maxhudson

I hope this will create a TypeScript definition also

codler avatar Apr 25 '20 17:04 codler