Skip to content

Update documentation on target framework updates#52855

Open
marcpopMSFT wants to merge 1 commit intomainfrom
marcpopMSFT-patch-2
Open

Update documentation on target framework updates#52855
marcpopMSFT wants to merge 1 commit intomainfrom
marcpopMSFT-patch-2

Conversation

@marcpopMSFT
Copy link
Member

Expanded the document to summarize the work required for updating target framework versions, including large updates and miscellaneous tasks. Added example PRs and a section for Net7 TFM updates.

Expanded the document to summarize the work required for updating target framework versions, including large updates and miscellaneous tasks. Added example PRs and a section for Net7 TFM updates.
Copilot AI review requested due to automatic review settings February 5, 2026 17:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR expands the documentation on updating target framework versions, making it more comprehensive and structured for future TFM updates. The changes provide a high-level summary of required work and preserve historical information about the Net7 update as a reference.

Changes:

  • Added a comprehensive summary section covering large updates (branding, KnownFrameworkReference, retargeting, test updates, templates) and miscellaneous tasks
  • Added example PRs from the net11 update to provide concrete references
  • Reorganized existing Net7 update information under a clearly labeled subsection


**Miscellaneous**
- Fix versions after the N-1 GA as there should be RC placeholders
- Likely will need to ping to N-1 templates
Copy link

Copilot AI Feb 5, 2026

Choose a reason for hiding this comment

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

The word "ping" should be "pin" in this context. The sentence should read "Likely will need to pin to N-1 templates."

Suggested change
- Likely will need to ping to N-1 templates
- Likely will need to pin to N-1 templates

Copilot uses AI. Check for mistakes.
Comment on lines +27 to +31
Example PRs from net11 update
[Backflow from VMR](https://github.com/dotnet/sdk/pull/52242)
[unwind test changes](https://github.com/dotnet/sdk/pull/52512)
[Branding](https://github.com/dotnet/sdk/pull/50468)
[BundledVersions](https://github.com/dotnet/sdk/pull/50329)
Copy link

Copilot AI Feb 5, 2026

Choose a reason for hiding this comment

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

This section header is inconsistently formatted. It should have a title format like "## Example PRs from net11 update" (with proper capitalization and consistent structure) or formatted as a list item. The following lines (28-31) should also be properly formatted as a bulleted list for consistency.

Suggested change
Example PRs from net11 update
[Backflow from VMR](https://github.com/dotnet/sdk/pull/52242)
[unwind test changes](https://github.com/dotnet/sdk/pull/52512)
[Branding](https://github.com/dotnet/sdk/pull/50468)
[BundledVersions](https://github.com/dotnet/sdk/pull/50329)
### Example PRs from net11 update
- [Backflow from VMR](https://github.com/dotnet/sdk/pull/52242)
- [unwind test changes](https://github.com/dotnet/sdk/pull/52512)
- [Branding](https://github.com/dotnet/sdk/pull/50468)
- [BundledVersions](https://github.com/dotnet/sdk/pull/50329)

Copilot uses AI. Check for mistakes.
- Recommend doing in the VMR and fixing tests on backflow
- We had to disable a bunch of tests to merge the branding flow
- [KnownFrameworkReference](https://github.com/dotnet/sdk/blob/main/src/Layout/redist/targets/BundledTemplates.targets) update
- Should be wrote but tricky to get right
Copy link

Copilot AI Feb 5, 2026

Choose a reason for hiding this comment

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

The word "wrote" should be "rote" (meaning a fixed, habitual, or mechanical course of procedure) in this context. The sentence should read "Should be rote but tricky to get right."

Suggested change
- Should be wrote but tricky to get right
- Should be rote but tricky to get right

Copilot uses AI. Check for mistakes.
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.

1 participant