Skip to content

[7.0-preview1] Update precedence of templates for 7.0 #39794

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

Merged
merged 3 commits into from
Jan 31, 2022
Merged

[7.0-preview1] Update precedence of templates for 7.0 #39794

merged 3 commits into from
Jan 31, 2022

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Jan 26, 2022

Contingent on validation of #39783 in the SDK

Update precedence of templates for 7.0

Description

Updates 7.0 template precedence so that the SDK doesn't confuse them with 6.0 templates

Customer Impact

Without this, customers can't use the 7.0 SDK to create 6.0 web apps (i.e. dotnet new web -f net6.0 doesn't work): #39731

Regression?

  • Yes (from 6.0)
  • No

[If yes, specify the version the behavior has regressed from]

Risk

  • High
  • Medium
  • Low

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

When servicing release/2.1

  • Make necessary changes in eng/PatchConfig.props

@wtgodbe wtgodbe requested a review from a team January 26, 2022 22:29
@Pilchie Pilchie added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Jan 26, 2022
@wtgodbe wtgodbe added the Servicing-approved Shiproom has approved the issue label Jan 28, 2022
@wtgodbe wtgodbe merged commit 116addb into dotnet:release/7.0-preview1 Jan 31, 2022
@wtgodbe wtgodbe deleted the wtgodbe/TemplatesP1 branch January 31, 2022 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates Servicing-approved Shiproom has approved the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants