-
Notifications
You must be signed in to change notification settings - Fork 71
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
Add typescript support #12
Comments
@reubenae Is this issue still available. If yes, I'd like to work on this. |
@nsinghal12 absolutely! We had some snagging issues on Adam's PR, which never got merged. You can refer to that for next steps if you get stuck. I recommend raising small, incremental PR's so we can avoid big merge conflicts. Looking forward to your contribution! |
Hi @skodamarthi is this issue apart of this year's hacktoberfest in addition to Grace Hopper? |
Hi @regromrob Yes...this is open for hacktoberfest as well. Contributions are welcome! |
Awesome @skodamarthi I would love to be assigned to this issue please. |
Hi! I actually ended up being really busy all of this month and won't have the time to tackle this ticket. I apologize for the inconvenience. Please unassign. Thank you! |
Hi @reubenae @skodamarthi Shall i pick this up ? |
I won't have access to my laptop for a couple of days so can't add an issue for the file migration - but absolutely happy for that to happen deliberately rather than over time. So if you fancy a crack at that, go ahead! Small PR's better than large ones though please! |
@pmaidens I'm back with laptop access now so feel free to make a PR and/or new issue as you wish, and I will review! |
This issue is being worked on, but there is lots of work to go around. If you're looking at this for some work, please keep your PR's small so that merge conflicts are avoided and wasted work limited. |
One of the most important things left for us to do here is that we need to publish our types for external consumption, and tie that into the exported functions to ensure we maintain those contracts. The other major piece left is to migrate tests to TypeScript as well. |
Is your feature request related to a problem? Please describe.
As an engineer, I want to be able to write my code in typescript, so I can find out possible bugs at compile time.
Describe the solution you'd like
Typescript being a supported language in this repo
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
This was attempted before (#28) but due to a series of merge conflicts, was closed without merging. I recommend approaching this in a piecemeal fashion. However, the previous PR is a good reference point for this work still.
The text was updated successfully, but these errors were encountered: