Skip to content

Update setTimeout and setInterval signatures #32190

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Update setTimeout and setInterval signatures #32190

wants to merge 1 commit into from

Conversation

yw662
Copy link

@yw662 yw662 commented Jul 1, 2019

#32186
The original signatures of setTimeout and setInterval may allow wrongly typed additional parameters passed to handler function.
This update fixes the issue without introducing any other changes
With this update, typescript can correctly reject wrongly typed additional parameters.

@fatcerberus is the author of the setTimeout signaure.

Fixes #

#32186 
The original signatures of `setTimeout` and `setInterval` may allow wrongly typed additional parameters passed to handler function.
This update fixes the issue without introducing any other changes
With this update, typescript can correctly reject wrongly typed additional parameters.

@fatcerberus is the author of the `setTimeout` signaure.
@typescript-bot
Copy link
Collaborator

It looks like you've sent a pull request to update our 'lib' files. These files aren't meant to be edited by hand, as they consist of last-known good states of the compiler and are generated from 'src'. Unless this is necessary, consider closing the pull request and sending a separate PR to update 'src'.

@typescript-bot
Copy link
Collaborator

It looks like you've sent a pull request to update some generated declaration files related to the DOM. These files aren't meant to be edited by hand, as they are synchronized with files in the TSJS-lib-generator repository. You can read more here. For house-keeping purposes, this pull request will be closed.

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.

2 participants