-
Notifications
You must be signed in to change notification settings - Fork 188
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
Declare Typescript custom typings #140
Comments
@pedrobranco What strategy do you want to work towards for this?
While I do not plan to completely solve the issue, I can begin the process to make it easier for other users to continue to build upon. |
@ruimarinho WDYT of using declaration file in |
How is anyone using TS at all in the Bitcoin ecosystem?
Another option not mentioned is using JSDoc. https://www.typescriptlang.org/docs/handbook/jsdoc-supported-types.html I'll add the JSDoc annotations since nothing exists and open a PR. I only hope it'll be accepted. 😆 I'll base it on #149 - This makes the types completely orthogonal. I've done the same with bitcoinjs/coinselect. I'll be opening a PR there as well. |
yes types would be nice, why not just add at least the types from index.d.ts in this old puill request #65 |
I guess we can go that way yeah. |
No description provided.
The text was updated successfully, but these errors were encountered: