Skip to content
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

Migrate project to TypeScript #86

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

monostere0
Copy link
Collaborator

🚨 Please review the guidelines for contributing to this repository.

Hi @ramgrandhi, peeps,

I've taken a bit of time to migrate this project to TypeScript. What's added is I've migrated all the files to TypeScript (shallowly for now).
The project does build and the tests do run. I have configured both eslint and jest to work with typescript as well.
One thing that remains is that we need to figure out how to build it before we publish it of course.

  • Updated unit tests (*.spec.js)
  • [] Updated e2e tests (here)
  • Updated documentation (here)

🙌 Thank you!

@ramgrandhi
Copy link
Owner

Awesome that you started it! 🤩

Let's pick a moment (after my holidays) and merge this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants