Skip to content

Conversation

@larabr
Copy link
Collaborator

@larabr larabr commented Nov 6, 2025

No description provided.

@larabr larabr force-pushed the ts-transform-types branch 2 times, most recently from 2007252 to a3fc34c Compare November 6, 2025 17:10
@larabr larabr requested a review from twiss November 7, 2025 10:48
@twiss twiss marked this pull request as draft November 7, 2025 11:54
@larabr larabr force-pushed the ts-transform-types branch 4 times, most recently from 723d708 to e32442f Compare November 12, 2025 13:09
@larabr larabr marked this pull request as ready for review November 12, 2025 13:21
@larabr larabr force-pushed the ts-transform-types branch from e32442f to 71deb41 Compare November 12, 2025 13:23
…d of parametric inference

With standalone declaration we cannot properly model the fact that `process`
and `finish` are never undefined due to the default values set in JS.
@larabr larabr force-pushed the ts-transform-types branch from d0f1d48 to 4029884 Compare November 13, 2025 13:11
larabr and others added 3 commits November 13, 2025 14:20
Cover minimum supported TS version.
… outputs

A downside of this solution is that we cannot enforce output type consistency
for the two callbacks since we need parametric inference of the respective
function types.
However, using such callbacks will now result in a union output type, that should signal
the usage might be off.
@larabr larabr force-pushed the ts-transform-types branch from 4029884 to 62cb334 Compare November 13, 2025 13:20
@larabr larabr merged commit 802b6b8 into main Nov 13, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants