-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
unpinning dependencies #26193
unpinning dependencies #26193
Conversation
@@ -4,6 +4,9 @@ | |||
<!-- MicrosoftNETCoreApp21PackageVersion is assigned at the bottom so it can automatically pick up MicrosoftNETCoreAppPackageVersion in an orchestrated build. --> | |||
<MicrosoftNETCoreAppPackageVersion>2.1.22</MicrosoftNETCoreAppPackageVersion> | |||
<MicrosoftNETCoreDotNetAppHostPackageVersion>2.1.22</MicrosoftNETCoreDotNetAppHostPackageVersion> | |||
<MicrosoftExtensionsCachingAbstractionsPackageVersion>2.1.2</MicrosoftExtensionsCachingAbstractionsPackageVersion> |
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.
@dougbu The Microsoft.Extension.Caching.Abstractions
and Microsoft.Extensions.Caching.Memory
packages are jumping from 2.1.2 to 2.1.23 in this patch (nuget.org has no intervening versions). Is this intentional? Shouldn't those be 2.1.3.
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.
Gaps are expected because Extensions doesn't build every assembly in every servicing release. Very likely these assemblies weren't updated since 2.1.2. They're definitely in 2.1.23. See https://github.com/dotnet/extensions/blob/dc1dab7c23212e095e280d202fa2151daa01e393/eng/PatchConfig.props#L14-L19
After the release, we'll probably need to repin all three of these packages at the versions produced in this release. Right @mmitche❔
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.
Yes correct. Just making sure that versioning is correct.
@@ -4,6 +4,9 @@ | |||
<!-- MicrosoftNETCoreApp21PackageVersion is assigned at the bottom so it can automatically pick up MicrosoftNETCoreAppPackageVersion in an orchestrated build. --> | |||
<MicrosoftNETCoreAppPackageVersion>2.1.22</MicrosoftNETCoreAppPackageVersion> | |||
<MicrosoftNETCoreDotNetAppHostPackageVersion>2.1.22</MicrosoftNETCoreDotNetAppHostPackageVersion> | |||
<MicrosoftExtensionsCachingAbstractionsPackageVersion>2.1.2</MicrosoftExtensionsCachingAbstractionsPackageVersion> |
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.
Gaps are expected because Extensions doesn't build every assembly in every servicing release. Very likely these assemblies weren't updated since 2.1.2. They're definitely in 2.1.23. See https://github.com/dotnet/extensions/blob/dc1dab7c23212e095e280d202fa2151daa01e393/eng/PatchConfig.props#L14-L19
After the release, we'll probably need to repin all three of these packages at the versions produced in this release. Right @mmitche❔
@dougbu Are these test failures expected? (This change shouldn't have any effect) |
Believe there are reasons why the |
No description provided.