Skip to content

Microsoft.Build: netstandard2.0 instead of netcoreapp2.0#3126

Merged
rainersigwald merged 4 commits intodotnet:vs15.7from
rainersigwald:target-netstandard2.0-and-netcoreapp2.1
Mar 23, 2018
Merged

Microsoft.Build: netstandard2.0 instead of netcoreapp2.0#3126
rainersigwald merged 4 commits intodotnet:vs15.7from
rainersigwald:target-netstandard2.0-and-netcoreapp2.1

Conversation

@rainersigwald
Copy link
Copy Markdown
Member

It's harder for projects to depend on us if we're netcoreapp2.0, because
they must themselves be .NET Core specific rather than netstandard.

We only need to be netcoreapp2.1, though--in 2.0 mode we can be
netstandard.

It's harder for projects to depend on us if we're netcoreapp2.0, because
they must themselves be .NET Core specific rather than netstandard.

We only need to be netcoreapp2.1, though--in 2.0 mode we can be
netstandard.
@rainersigwald rainersigwald added this to the MSBuild 15.7 milestone Mar 23, 2018
@rainersigwald
Copy link
Copy Markdown
Member Author

rainersigwald commented Mar 23, 2018

<TargetFrameworks>$(RuntimeOutputTargetFrameworks)</TargetFrameworks>
<!-- Node reuse requires an API new to .NET Core 2.1 not yet available
in .NETStandard. -->
<TargetFrameworks>netstandard2.0;netcoreapp2.1</TargetFrameworks>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

How does this not break our netcoreapp2.0 target? I assume that one's also getting tested, right?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The netcoreapp2.0 target uses the netstandard2.0 library from here.

@rainersigwald rainersigwald merged commit b5dc4cf into dotnet:vs15.7 Mar 23, 2018
@rainersigwald rainersigwald deleted the target-netstandard2.0-and-netcoreapp2.1 branch March 23, 2018 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants