Skip to content

Conversation

@directhex
Copy link
Contributor

Resolves official build breakage caused by #74428 (where PackageRID can be linux-wasm or osx-wasm instead of browser-wasm when /p:PortableBuild=true, but OutputRid is fine)

@directhex
Copy link
Contributor Author

@radical
Copy link
Member

radical commented Jan 6, 2023

/azp run runtime-wasm-non-libtests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@directhex
Copy link
Contributor Author

@radical the one failed test seems unrelated. Can we slap an approval on this to get it merged and un-break official builds?

@radical
Copy link
Member

radical commented Jan 7, 2023

@radical the one failed test seems unrelated. Can we slap an approval on this to get it merged and un-break official builds?

Yep! I would just wait for the dotnet-runtime-official build to complete.

Copy link
Member

@radical radical left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@radical radical merged commit 58614a0 into dotnet:main Jan 7, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Feb 6, 2023
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.

2 participants