Skip to content

[wasm][bindings] Fix an api regression introduced in #42486#42658

Merged
lewing merged 2 commits intodotnet:masterfrom
lewing:wasm-null-js
Sep 24, 2020
Merged

[wasm][bindings] Fix an api regression introduced in #42486#42658
lewing merged 2 commits intodotnet:masterfrom
lewing:wasm-null-js

Conversation

@lewing
Copy link
Copy Markdown
Member

@lewing lewing commented Sep 23, 2020

Return null from js_string_to_mono_string when passed null, like it did prior to #42486

fixed broken wasm debugger tests that need to run in CI #42190

@lewing lewing requested a review from marek-safar as a code owner September 23, 2020 23:53
@lewing lewing requested review from kg and radical September 23, 2020 23:54
@lewing lewing requested a review from thaystg September 24, 2020 00:05
@lewing lewing changed the title Fix an api regression introduced in #42486 [wasm][bindings]Fix an api regression introduced in #42486 Sep 24, 2020
@lewing lewing changed the title [wasm][bindings]Fix an api regression introduced in #42486 [wasm][bindings] Fix an api regression introduced in #42486 Sep 24, 2020
Comment thread src/mono/wasm/runtime/binding_support.js Outdated
a number it is

Co-authored-by: Ankit Jain <radical@gmail.com>
kg added a commit to kg/runtime that referenced this pull request Sep 24, 2020
@lewing lewing merged commit 117b18a into dotnet:master Sep 24, 2020
github-actions bot pushed a commit that referenced this pull request Sep 30, 2020
marek-safar pushed a commit that referenced this pull request Oct 9, 2020
)

Co-authored-by: Katelyn Gadd <kg@luminance.org>
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants