Skip to content
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

Fix failing official build #80316

Merged
merged 1 commit into from
Jan 7, 2023

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

Official build validation at https://dev.azure.com/dnceng/internal/_build/results?buildId=2081718&view=results - hopefully my fix didn't break other things

@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