Add /** @deferred */ comments to deferred callback parameters#1757
Add /** @deferred */ comments to deferred callback parameters#1757ahejlsberg wants to merge 4 commits intomainfrom
Conversation
|
Thanks for the PR! This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged. |
|
You also need to |
|
(Looks good. feel free to merge when the TS PR is ready) |
|
|
|
And another question, is it the requirement here for the callback to be called asynchronously? What happens if those callbacks can be called both sync/async? |
The BTW, I haven't been able to find any authoritative documentation on sync vs. async for the various DOM callbacks, so it could well be that some of the modifiers shouldn't be present. |
This PR adds
/** @deferred */comments to deferred callback parameters in preparation for microsoft/TypeScript#58729.