Skip to content

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Mar 25, 2020

I didn't follow https://github.com/dotnet/aspnetcore-internal/blob/master/docs/engineering/servicing-and-preview-fixes.md after today's difficulties - will make sure the merge bot gets this up into master

@wtgodbe wtgodbe requested review from JunTaoLuo and dougbu March 25, 2020 23:41
@@ -60,4 +60,8 @@ Later on, this will be checked using this condition:
<PackagesInPatch>
</PackagesInPatch>
</PropertyGroup>
<PropertyGroup Condition=" '$(VersionPrefix)' == '2.1.18' ">
Copy link
Contributor

Choose a reason for hiding this comment

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

FYI it's fine to reuse an empty <PropertyGroup> for the previous servicing release. But, not worth further changes.

@Pilchie Pilchie added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Mar 26, 2020
@wtgodbe
Copy link
Member Author

wtgodbe commented Mar 26, 2020

CI is green but hasn't yet reported

@wtgodbe wtgodbe merged commit 2604ce9 into release/2.1 Mar 26, 2020
@wtgodbe wtgodbe deleted the wtgodbe/2118 branch March 26, 2020 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants