Add Typescript definition file for the grpc-web npm package

[aberasarte]

Now that the grpcweb runtime is available as an npm package 🎉 🎉 , it would be even better if the package could include a Typescript definition file as well. This way, it would be much easier to consume the grpcweb runtime from Typescript code.

If this isn't feasible I'm thinking about submitting a d.ts file to definitely typed but this is not the best solution.

[stanley-cheung]

We definitely have plan to add an index.d.ts file to the npm package. I tried to use an internal version of the Angular Clutz tool to generate the .d.ts typings - it sort-of works but the output is just a bunch of any and it has these declare namespace ಠ_ಠ.clutz { thing.

The grpc node package hand-writes their index.d.ts file so I might have to do that and it might take a while.

Does anybody know of any other closure to typescript typings generator I can try? Or is there someone with more Typescript experience who can start a general structure of the desired index.d.ts file and I can certainly contribute to the rest of the content? Contributions are definitely welcome!

[jonahbron]

I might be able to contribute a new file with the basic outline this week.

[glerchundi]

This would be fantastic!

…

On Tue, 21 Aug 2018 at 20:42, Jonah Dahlquist ***@***.***> wrote: I might be able to contribute a new file with the basic outline this week. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#255 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACIPluKO1SEuyR19PjUjjOaMtBC2J_F6ks5uTFSbgaJpZM4WFz4o> .

[stanley-cheung]

I started a PR #258 to show how the index.d.ts might work. Any feedback appreciated!

[stanley-cheung]

PR #258 merged. The index.d.ts typings should be making it into the next release 0.4.0 currently planned for next week.

After that user should be able to do import * as grpcWeb from 'grpc-web'; after an npm i grpc-web.

[aberasarte]

Great! Thanks for adding this @stanley-cheung .

[stanley-cheung]

The npm package version 0.4.0 has been pushed. Please try it out!