Skip to content

Conversation

@RReverser
Copy link
Contributor

@RReverser RReverser commented Oct 2, 2019

This bumps generated bindings to include microsoft/TypeScript-DOM-lib-generator#767.

Note that other related APIs were already updated in microsoft/TypeScript-DOM-lib-generator#755 and microsoft/TypeScript-DOM-lib-generator#756 and included in current TypeScript beta.

Unfortunately, this one slipped under the radar, but it would be really great to include these changes in beta too for consistency between different WebAssembly methods.

This bumps generated bindings to include microsoft/TypeScript-DOM-lib-generator#767.

Note that other related APIs were already updated in microsoft/TypeScript-DOM-lib-generator#755 and microsoft/TypeScript-DOM-lib-generator#756 and included in current TypeScript beta.

Unfortunately, this one slipped under the radar, but it would be really great to include these changes in beta too for consistency between different WebAssembly methods.
@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.

@RReverser
Copy link
Contributor Author

sigh

@sandersn @DanielRosenwasser @orta can you reopen this please? I did copy these files from TSJS-lib-generator, and made a PR here as suggested before, to get changes into TypeScript in time.

@DanielRosenwasser
Copy link
Member

Adding @RyanCavanaugh. This is technically breaking, but I would guess it would be relatively limited in impact for 3.7 RC.

@RReverser
Copy link
Contributor Author

@DanielRosenwasser Thanks!

@sandersn sandersn self-assigned this Oct 3, 2019
@sandersn sandersn merged commit 1820df1 into microsoft:master Oct 14, 2019
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants