-
-
Notifications
You must be signed in to change notification settings - Fork 54
Run dedicated typescript types tests using tsd #363
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
base: master
Are you sure you want to change the base?
Conversation
|
@jankapunkt Can I ask why did you pick Only my opinions, here is how TSTyche could be helpful for your project:
Might be you have your own arguments and the decision is made already. If so, sorry about the noise. |
|
@mrazauskas thank you for the input. I chose tsd mostly because the famous unicorn guy with the puppy dog is working on it, haha. Addtionally I like the ease of use and convention over configuration. I also looked at your repo and it looks future proof. Maybe @shrihari-prakash or @dhensby have also an opinion on this one |
| { | ||
| // Visit https://aka.ms/tsconfig to read more about this file | ||
| "compilerOptions": { | ||
| "extends": "@tsconfig/node16/tsconfig.json", |
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.
Hm.. extends must be a sibling of compilerOptions. It is not a compiler option. Really strange tsd does not complain about that.
Indeed that is good. There are also other nice aspects of In contrary, TSTyche would select all I do like many aspects of (Why didn’t I try to improve |
Summary
Due to multiple conflicts with the type signatures in our Request object, I started a little spec for types testing using
tsd.Please note, that I added dev dependencies (express, undici) for now. I am currently unsure if this is good in the long-run and we may rather extract these into the specific repos for express, koa etc.
Linked issue(s)
#356 #362
Involved parts of the project
dev-only, tests, typescript
Added tests?
yes
OAuth2 standard
not involved here
Reproduction