Clarify dotnet format is built into .NET SDK and remove SETUP.md reference#56
Merged
Chris-Wolfgang merged 5 commits intodevelopfrom Feb 2, 2026
Merged
Conversation
Owner
|
@copilot retry working on this PR |
Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>
Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>
Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>
Contributor
Author
Copilot
AI
changed the title
[WIP] WIP Address feedback from review on develop pull request
Clarify dotnet format is built into .NET SDK and remove SETUP.md reference
Feb 2, 2026
Chris-Wolfgang
approved these changes
Feb 2, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR clarifies documentation and tooling around the dotnet format command by explaining that it has been built into the .NET SDK since version 6.0 and requires no separate installation. The changes improve developer experience by removing broken documentation references and adding helpful error messages.
Changes:
- Added Prerequisites section to formatting documentation explaining
dotnet formatis built into .NET 6+ SDK - Enhanced
format.ps1script with preflight check to verifydotnet formatavailability - Fixed broken
SETUP.mdreference in README by replacing it with link toCONTRIBUTING.md
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| README-FORMATTING.md | Added Prerequisites section clarifying dotnet format is built-in and deprecated standalone tool |
| format.ps1 | Added preflight validation check with helpful error messaging for missing .NET SDK |
| README.md | Replaced broken SETUP.md link with valid CONTRIBUTING.md reference |
| .github/workflows/pr.yaml | Added inline comment documenting that dotnet format is built into SDK |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot stopped work on behalf of
Chris-Wolfgang due to an error
February 2, 2026 23:41
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR addresses review feedback on PR #51 regarding
dotnet formattool installation and documentation references. The changes clarify that the standalonedotnet-formattool was deprecated whendotnet formatwas integrated into the .NET 6 SDK in August 2021, and removes a broken documentation link.Original PR: #51
Triggering review: #51 (comment)
Type of change
How Has This Been Tested?
format.ps1script with preflight check to verify it works correctlydotnet formatcommand works as expected without separate installationChecklist
Screenshots (if applicable)
N/A - Documentation and script changes only
Additional context
Changes Made:
README-FORMATTING.md - Added a Prerequisites section explaining that
dotnet formatis built into .NET 6+ SDK and no separate tool installation is neededformat.ps1 - Added a preflight check that verifies
dotnet formatis available and provides helpful error messaging if the SDK is not installed.github/workflows/pr.yaml - Added a comment clarifying that
dotnet formatis built into the .NET 6+ SDKREADME.md - Removed the broken reference to
SETUP.mdand replaced it with a link toCONTRIBUTING.mdSecurity Summary
No vulnerabilities were introduced or discovered. CodeQL security scan passed with 0 alerts.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.