-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add dependencies needed for source-build PVP flow #33039
Conversation
eng/Versions.props
Outdated
<NuGetPackagingPackageVersion>6.7.0-preview.2.51</NuGetPackagingPackageVersion> | ||
<NuGetVersioningPackageVersion>6.7.0-preview.2.51</NuGetVersioningPackageVersion> | ||
<NuGetPackagingVersion>6.7.0-preview.2.51</NuGetPackagingVersion> | ||
<NuGetProjectModelVersion>6.7.0-preview.2.51</NuGetProjectModelVersion> | ||
</PropertyGroup> | ||
<PropertyGroup> | ||
<!-- Dependencies from https://github.com/Microsoft/vstest --> | ||
<MicrosoftNETTestSdkPackageVersion>17.7.0-preview.23302.1</MicrosoftNETTestSdkPackageVersion> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the version is explicitly used further down, do we still need the property?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MicrosoftNETTestSdkPackageVersion
is used in various places in this repo, we need this property.
Contributes to dotnet/source-build#3043
PVP flow is a method of flowing dependencies in source build that only overrides versions of packages that the repo has a declared dependency on in
Version.Details.xml
. This means that the package flow is closer to the Microsoft's way of building .NET.To enable this for
sdk
, we just need to add dependencies on the packages that should be overridden in source build. In this case, it's the variousnuget.client
and 2vstest
packages. I've also updatedVersions.props
file to remove usage of proxy properties for these repos - this is needed for proper PVP flow.To fully enable PVP flow, the repo project in the VMR will needto set
<PackageVersionPropsFlowType>DependenciesOnly</PackageVersionPropsFlowType>
once these changes flow in.