Skip to content
This repository was archived by the owner on Mar 26, 2025. It is now read-only.

Move templates to target net8.0#105

Merged
dougbu merged 1 commit into
dotnet:mainfrom
dougbu:dougbu/net8.0
Jan 5, 2023
Merged

Move templates to target net8.0#105
dougbu merged 1 commit into
dotnet:mainfrom
dougbu:dougbu/net8.0

Conversation

@dougbu
Copy link
Copy Markdown
Contributor

@dougbu dougbu commented Dec 17, 2022

  • add loc comments to ensure 'net8.0' doesn't become 'Net8.0', '.Net 8.0', ...
  • bump template precedence and identity values

- add loc comments to ensure 'net8.0' doesn't become 'Net8.0', '.Net 8.0', ...
- bump template precedence and identity values
@dougbu dougbu requested a review from a team December 17, 2022 21:26
Copy link
Copy Markdown
Contributor Author

@dougbu dougbu left a comment

Choose a reason for hiding this comment

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

Most of this matches the template changes made in dotnet/aspnetcore#45475. I commented on the exceptions.

Comment thread src/content/Angular-CSharp/.template.config/template.json
@dougbu dougbu added the area-infrastructure Includes: MSBuild projects/targets, build scripts, pipeline support label Dec 19, 2022
@dougbu
Copy link
Copy Markdown
Contributor Author

dougbu commented Jan 3, 2023

/ping reviewers

Copy link
Copy Markdown
Member

@wtgodbe wtgodbe left a comment

Choose a reason for hiding this comment

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

LGTM other than the 2 questions

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, pipeline support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants