-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Setup TS testing infrastructure #4659
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me! thanks
package.json
Outdated
@@ -103,6 +103,7 @@ | |||
"@types/express": "^4.17.14", | |||
"@types/fs-extra": "^9.0.13", | |||
"@types/jquery": "^3.5.10", | |||
"@types/mocha": "^10.0.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's a way newer version than the mocha we're using, but I guess it works ok anyway: our mocha is stuck in the past because of the ESM issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, @types/xyz
versions don't always correspond to xyz
's repositories so I wasn't sure what to do. At least for the file I converted the types seemed fine. I can try throwing a 8.x.x version on and see.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy to approve this, did you try out mocha-ts? (Maybe this is easier to run than mocha-ts)
Thanks @matsjla, I haven't tried out |
Okay sounds great! |
(and tsify test/ansi-to-html-tests at the same time)