Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

updating assembly versions, packageversions and publishing the packages#42823

Merged
Anipik merged 3 commits intodotnet:release/3.1from
Anipik:packageUpdate
Jan 15, 2020
Merged

updating assembly versions, packageversions and publishing the packages#42823
Anipik merged 3 commits intodotnet:release/3.1from
Anipik:packageUpdate

Conversation

@Anipik
Copy link

@Anipik Anipik commented Jan 15, 2020

We have serviced 6 commits in 3.1.2
3 -> system.data.sqlclient
1 -> system.text.json
1 -> Reflection.MetadataLoadContext (eric already made that change for this change)
1 -> changes system.Native (part of shared framework)

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.

Could you also update System.Drawing.Common to contain 4.6.2 to fix harvesting validation?

I did it in runtime in master, it needs to be done here as well: dotnet/runtime#1729

@Anipik Anipik merged commit bbcc0a8 into dotnet:release/3.1 Jan 15, 2020
@Anipik Anipik deleted the packageUpdate branch January 15, 2020 06:32
"InboxOn": {
"netcoreapp3.0": "4.0.0.0",
"netcoreapp3.1": "4.0.1.0",
"netcoreapp3.1": "4.0.1.1",
Copy link
Member

Choose a reason for hiding this comment

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

For future reference, this change was technically wrong as this assembly is also shipping out of an OOB package. When that is the case, you want to keep the InboxOn version frozen to the version you first shipped it on that framework, that way, if you ever service any package from this branch that takes a dependency on this library, our packaging infrastructure will force adding a package reference to the serviced version of this package.

cc: @Anipik @ericstj

Copy link
Member

Choose a reason for hiding this comment

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

In all cases InboxOn should be frozen. Once an assembly is inbox in a framework that's defined the minimum version provided by the framework. Since a framework is targeted with a single thing: the TFM, we cannot depend on anything but the minimum version.

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.

5 participants