Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Improve WebAssembly.instantiateStreaming#756

Merged
orta merged 1 commit into
microsoft:mastermicrosoft/TypeScript-DOM-lib-generator:masterfrom
RReverser:improve-wasm-imports-exportsRReverser/TSJS-lib-generator:improve-wasm-imports-exportsCopy head branch name to clipboard
Sep 23, 2019
Merged

Improve WebAssembly.instantiateStreaming#756
orta merged 1 commit into
microsoft:mastermicrosoft/TypeScript-DOM-lib-generator:masterfrom
RReverser:improve-wasm-imports-exportsRReverser/TSJS-lib-generator:improve-wasm-imports-exportsCopy head branch name to clipboard

Conversation

@RReverser

Copy link
Copy Markdown
Contributor

Follow-up to #755.

@RReverser

Copy link
Copy Markdown
Contributor Author

@orta Apologies, this should be the last one.

@orta

orta commented Sep 23, 2019

Copy link
Copy Markdown
Contributor

👍

@orta orta merged commit a2d6aeb into microsoft:master Sep 23, 2019
@RReverser

Copy link
Copy Markdown
Contributor Author

@orta thanks :) To duplicate my question from the last PR:

@orta Thanks! Do you know if I need to do anything further to get definitions updated on the TypeScript side or are they pulled from here automatically on new release?

@RReverser RReverser deleted the improve-wasm-imports-exports branch September 23, 2019 21:24
@orta

orta commented Sep 23, 2019

Copy link
Copy Markdown
Contributor

Hrm, it looks to me like these changes are then PR'd into the TypeScript repo (maybe this was the last PR doing that - microsoft/TypeScript#32335 )

@RReverser

Copy link
Copy Markdown
Contributor Author

@orta I see. Do you want / need to do that as a maintainer, or should I?

@RReverser

Copy link
Copy Markdown
Contributor Author

@orta Sorry, but ping :) Should I do something next?

@orta

orta commented Sep 25, 2019

Copy link
Copy Markdown
Contributor

Yep, I’d recommend sending a change over to the main repo - otherwise it’ll happen whenever we do an update ourselves

sandersn pushed a commit to microsoft/TypeScript that referenced this pull request Oct 14, 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.
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.