-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[release/8.0] Calculate PackageVersionNet7 #94614
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
[release/8.0] Calculate PackageVersionNet7 #94614
Conversation
Since 8.0 is GA, we'll be able to calculate the net7 version based on PatchVersion
|
Tagging subscribers to this area: @dotnet/area-infrastructure-libraries Issue DetailsSince 8.0 is GA, we'll be able to calculate the net7 version based on PatchVersion
|
|
@carlossanlop this is an infra only change. |
| <PatchVersion>0</PatchVersion> | ||
| <SdkBandVersion>8.0.100</SdkBandVersion> | ||
| <PackageVersionNet7>7.0.12</PackageVersionNet7> | ||
| <PackageVersionNet7>7.0.$([MSBuild]::Add($(PatchVersion),14))</PackageVersionNet7> |
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.
If we do not ship 6 and 7 along side 8 in the next release this will need to change
| <PackageVersionNet7>7.0.$([MSBuild]::Add($(PatchVersion),14))</PackageVersionNet7> | |
| <PackageVersionNet7>7.0.$([MSBuild]::Add($(PatchVersion),13))</PackageVersionNet7> |
|
@steveisok can you please take a look at the feedback? Today's code complete. |
|
We should merge this and follow up if necessary. |
Since 8.0 is GA, we'll be able to calculate the net7 version based on PatchVersion