Skip to content

Conversation

@mmitche
Copy link
Member

@mmitche mmitche commented Sep 15, 2021

Before this was not important because runtime was calling its own publishing tasks with the property explicitly set, but since the introduction of arcade powered source build this property is needed to avoid ArPow from defaulting to v2.

Before this was not important because runtime was calling its own publishing tasks with the property explicitly set, but since the introduction of arcade powered source build this property is needed to avoid ArPow from defaulting to v2.
@ghost
Copy link

ghost commented Sep 15, 2021

Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

Before this was not important because runtime was calling its own publishing tasks with the property explicitly set, but since the introduction of arcade powered source build this property is needed to avoid ArPow from defaulting to v2.

Author: mmitche
Assignees: -
Labels:

area-Infrastructure-libraries

Milestone: -

@mmitche
Copy link
Member Author

mmitche commented Sep 15, 2021

Should fix build break due to #59108.

@mmitche
Copy link
Member Author

mmitche commented Sep 15, 2021

/cc @radical

@mmitche
Copy link
Member Author

mmitche commented Sep 15, 2021

@mmitche
Copy link
Member Author

mmitche commented Sep 15, 2021

I'm gonna merge this so its gets in before the snap. The changes have no effect on the testing

@mmitche mmitche merged commit b83e5fe into dotnet:main Sep 15, 2021
Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

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

LGTM

@mmitche mmitche deleted the set-publishing-infra-version-main branch September 20, 2021 21:17
@ghost ghost locked as resolved and limited conversation to collaborators Nov 3, 2021
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