-
Notifications
You must be signed in to change notification settings - Fork 25
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
Move the Cypress tests to TypeScript #18
Comments
Pull request is already merged so could you close this issue? :) |
Hi @Lukaszli , the closed PR was "update the TS definition" while this one is "translate the existing cypress tests to TS" so, no, it could not be closed yet 😊 |
HI @SteveSchreiner, thank you 😊 |
The #66 could have been prevented by converting all the tests to TypeScript. |
Cypress released |
Today I imported this plugin into a TypeScript-based Cypress project. TS did not like the old definition (that's why I fixed it with #17 ) but we must avoid that the end user discovers it himself.
That's why I propose to move the Cypress test to TypeScript. @allevo do you want to do that? 👋
The text was updated successfully, but these errors were encountered: