-
Notifications
You must be signed in to change notification settings - Fork 14
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
Translate library to TypeScript #51
Comments
Hey @spencerudnick, yeah please feel free to do so. Also: can we make the types available somehow for consumers of the lib, do you have any idea on how that could work? |
Can you sync it to the next-release one? (open a PR to merge into it) Only use TypeScript for the actual typing. |
I have a tsconfig setup in another library, which builds to ES5 and
includes type definitions along with each module.
I will include that in the PR tonight 😊
…On Tue, Jan 22, 2019, 8:18 AM Geoffrey Dhuyvetters ***@***.*** wrote:
Hey @spencerudnick <https://github.com/spencerudnick>, yeah please feel
free to do so.
Also: can we make the types available somehow for consumers of the lib, do
you have any idea on how that could work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#51 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AC7if_LgukONwEbFCdyY47TOh0rQLXigks5vFx2vgaJpZM4aMwng>
.
|
If you prefer to use Babel with Typescript, I think all that needs to be
done is to target `esnext` during TS transpilation.
On Tue, Jan 22, 2019, 12:10 PM Spencer Rudnick <[email protected]
wrote:
… I have a tsconfig setup in another library, which builds to ES5 and
includes type definitions along with each module.
I will include that in the PR tonight 😊
On Tue, Jan 22, 2019, 8:18 AM Geoffrey Dhuyvetters <
***@***.*** wrote:
> Hey @spencerudnick <https://github.com/spencerudnick>, yeah please feel
> free to do so.
>
> Also: can we make the types available somehow for consumers of the lib,
> do you have any idea on how that could work?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#51 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AC7if_LgukONwEbFCdyY47TOh0rQLXigks5vFx2vgaJpZM4aMwng>
> .
>
|
I've started with the conversion based on the link you provided, nice way to develop with TypeScript without using it to compile. I'm noticing some redundancies after the conversion, so I'll fix those up first. |
Hey guys, I'm interested in a conversion happening. I've used TSDX in a bunch of projects, might be a good fit here. https://github.com/formium/tsdx |
I might do a big refactor in the future. I always had the idea of creating an internal circle of fifths and take everything from there. Interested in joining? |
@duivvv Thanks for your note about collaborating. I'm opening this issue to track the work.
I have done some work on my fork. With your permission, I'll merge these changes to a branch on this repository for testing and review.
The text was updated successfully, but these errors were encountered: