Change never type to unknown#4142
Conversation
Using `never` was a workaround I introduced in 3002, because `unknown` wasn’t supperted by all relevant versions of TypeScript at the time. It is now, and it is much more correct.
|
Oh, my understanding was that introducing |
|
That is still being enforced by this, it’s just a better but still stupid solution |
|
@ImRodry Please have a look at the code of conduct and reconsider your wording. Besides, could you please elaborate on why specifying the type might not always be desired when using axios in a TypeScript project, and possibly provide an example? |
That was exactly my goal, but using You can see the difference by hovering over the values inside the if-statement blocks in this playground link |
Sure I’ll give you the exact example I’m working with |
|
@ImRodry What stops you from passing |
Please stop suggesting any as I already explained the issue with that from a user’s side countless times |
|
@ImRodry I think we're not on the same page here, so I would suggest you to create an issue with a complete and more specific description of the problem you're having (including a code snippet). Besides, my advice would be to keep your wording friendly, if you actually want maintainers to invest their time in your issue. Maybe you're right, but calling a change you don't agree with "stupid" or "useless" will probably not get you what you want. |
|
I have. Please see #4141 |
Using
neverwas a workaround I introduced in 3002, becauseunknownwasn’t supperted by all relevant versions of TypeScript at the time. It is now, and it is much more correct.