-
Notifications
You must be signed in to change notification settings - Fork 167
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
Where is the love for typescript? #101
Comments
Pull requests are welcomed :)
…On Sat, Dec 22, 2018, 08:01 P0oOOOo0YA ***@***.*** wrote:
Loved the project but sadly there is no type definition file available.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#101>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AANMjKSXV_8_JjUTc_BBY-MAAFUA6jfgks5u7ldngaJpZM4ZfhYa>
.
|
I took a look at making a TS Definition file but can't understand the format of the invoke method, it looks highly irregular and therefore it might not be possible to create TS definitions. |
It is my understanding that typescript is a superset of JavaScript. It
should technically be possible to typecheck the existing invoke function.
And what about adding new functions that are more ergonomic for typescript?
Would that resolve your issue?
…On Mon, Oct 28, 2019, 16:21 Harris Mirza ***@***.***> wrote:
I took a look at making a TS Definition file but can't understand the
format of the invoke method, it looks highly irregular and therefore it
might not be possible to create TS definitions.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#101?email_source=notifications&email_token=AABUZDFFYKOBZAD7AJEXPJ3QQ5XW3A5CNFSM4GL6CYNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECOWZ2A#issuecomment-547187944>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABUZDE3MC2V4GKGSPD6YLDQQ5XW3ANCNFSM4GL6CYNA>
.
|
The main issue I could see without looking in too much detail was that the args parameter in Invoke isn't at the end which is a requirement for 'rest parameters'. If changing the API isn't out of the question, I think it would make more sense to have that at the end anyway and move the callback to before the arguments. |
Loved the project but sadly there is no type definition file available.
The text was updated successfully, but these errors were encountered: