Skip to content
This repository was archived by the owner on Dec 20, 2018. It is now read-only.

[2.1]: Add package version to Microsoft.AspNetCore.App in project templates #603

Merged
merged 2 commits into from
Jul 3, 2018

Conversation

natemcmaster
Copy link
Contributor

Part of dotnet/aspnetcore#3292

We're reversing course on the implicit package version feature. This adds a package version for Microsoft.AspNetCore.App to the dotnet new and VS templates.

cc @DamianEdwards @JunTaoLuo

@natemcmaster natemcmaster added the servicing-consider Shiproom approval is required for the issue label Jul 3, 2018
@natemcmaster natemcmaster requested a review from JunTaoLuo July 3, 2018 01:25
Copy link

@JunTaoLuo JunTaoLuo left a comment

Choose a reason for hiding this comment

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

Changes look good. Have we gotten approval from shiproom for this yet? There would be quite a few related PRs to completely address this problem right?

$@"<Project>
<Import Project=""{templatesTestsTargetsFilePath}"" />
</Project>";
const string directoryBuildTargetsContent = "<Project></Project>";

Choose a reason for hiding this comment

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

nit:

File.WriteAllText(Path.Combine(TemplateOutputDir, "Directory.Build.targets"), "<Project />");
would be sufficient here.

@natemcmaster natemcmaster added servicing-approved Shiproom has approved the issue and removed servicing-consider Shiproom approval is required for the issue labels Jul 3, 2018
@natemcmaster
Copy link
Contributor Author

Yes, this was approved in shiproom this morning. We will be making a bunch of tweaks to docs, warnings, tooling, etc. over the next few weeks to follow up on this change.

@natemcmaster natemcmaster merged commit 727bd52 into release/2.1 Jul 3, 2018
@natemcmaster natemcmaster deleted the namc/pkg-version branch July 3, 2018 23:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
servicing-approved Shiproom has approved the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants