Conversation
|
It's definitely not desirable - I've been looking at it and it seems like #51580 and #52837 conflict semantically, to the tune of partially re-breaking what #52837 fixed. (Also the In the meantime, we can revert #51580 to put |
|
(Specifically, by no longer passing in the |
|
Sent #53283. |
| <Foo {...{ | ||
| a: (x) => 10, | ||
| b: (arg) => { arg.toString(); }, | ||
| ~~~ | ||
| !!! error TS18046: 'arg' is of type 'unknown'. | ||
| }} />; |
There was a problem hiding this comment.
From what I see this is the only "unwanted" change that has been found on main. However... this is exactly how regular call expressions work: TS playground. So perhaps... this was just working accidentally before. I feel like this should be a separate issue to make it work for both JSX and regular calls (if that's desired)
I took a look at those changes and I think that some of them are not desired. |
#51580 had not been updated in a while and has some baseline changes.
I am not sure the behavior of
intraExpressionInferencesJsxis desirable here. @Andarist