Run dedicated typescript types tests using tsd#363
Run dedicated typescript types tests using tsd#363jankapunkt wants to merge 4 commits intomasterfrom
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.
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 |
…o tests/typescript-checks
|
Blocked until #383 is merged |
|
#383 is merged, this is free for review now |
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