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

Quote property value #52917

Merged
merged 1 commit into from
May 18, 2021
Merged

Conversation

am11
Copy link
Member

@am11 am11 commented May 18, 2021

@ViktorHofer, @akoeplinger sorry missed checking all the legs logs 😄

@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ViktorHofer
Copy link
Member

Merging to unblock failing CI legs.

@ViktorHofer ViktorHofer merged commit c5caf17 into dotnet:main May 18, 2021
janvorli added a commit to janvorli/runtime that referenced this pull request May 19, 2021
Port of dotnet#52799 and dotnet#52917

During the repo consolidation, we missed passing OfficialBuildId
argument to common script from all subsets, which resulted in default
version emitted in the binaries produced by Libs subset in the official
builds.
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
Anipik pushed a commit that referenced this pull request Jun 3, 2021
* [Release/5.0] - Fix versions for Libs native artifacts

Port of #52799 and #52917

During the repo consolidation, we missed passing OfficialBuildId
argument to common script from all subsets, which resulted in default
version emitted in the binaries produced by Libs subset in the official
builds.

* Fix empty build version case
@ghost ghost locked as resolved and limited conversation to collaborators Jun 19, 2021
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