-
Notifications
You must be signed in to change notification settings - Fork 149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TypeScript Support #128
Comments
Typings would definitely be welcome. I doubt I'll have time to dig into TypeScript enough to add them anytime soon unfortunately; I'll keep this open in case someone else would like to take a stab at it. Thank you for the kind words and bringing this up! |
I have an incomplete version here: https://github.com/andrew8er/avsc/tree/feature/typescript-definitions I am pretty sure there are some errors, since I'm only really using a small part of the API so far. |
Thanks @andrew8er! Can you submit a PR with what you have so far? |
Merged - thanks @andrew8er. Keeping this open since the typing file isn't complete yet. |
What's the intent for the package https://www.npmjs.com/package/@avro/types ?
Also, publishing declarations that are neither in "@types/avsc" nor packaged with "avsc" is non-standard and surprising. I proposed #146 ("bundling with your npm package") , but closed it since I'm not sure what the long term plan is. According to https://github.com/Microsoft/TypeScript-Handbook/blob/master/pages/declaration%20files/Publishing.md
|
Hey @TysonAndre, thank you for the publishing tips. I'd created the |
Any status on this work? It's a really terrific initiative so I'm not nagging, just excited! |
@yonkeltron - still a WIP but we already have reasonably complete typings thanks to @andrew8er and @TysonAndre. |
Roger! I'll pass it along to my teammates and we can try it out. Thanks for the hard work! |
Just added a pull request to improve the typings... |
Tweak typings + Make options in `forProtocol` optional + Add optional transport to `ClientOptions`
shameless plug: you can see how I use it in TS here: https://github.com/RandomFractals/vscode-data-preview/blob/master/src/data.providers/avro.data.provider.ts |
I apologize for commenting without being able to dedicate time to opening a PR, but wanted to point out this really handy set of features that live in AJV: https://ajv.js.org/guide/typescript.html In particular, it would be amazing if:
I hope these ideas make sense |
Thanks for the hard work on this - great project.
Any plans to include type definitions for TypeScript?
The text was updated successfully, but these errors were encountered: