-
Notifications
You must be signed in to change notification settings - Fork 497
Update dotnet package commands to noun-verb format #3506
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
base: main
Are you sure you want to change the base?
Conversation
|
Note: I did not update release notes |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
There was a problem hiding this 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 updates dotnet CLI command syntax throughout the documentation from the old format (e.g., dotnet add package) to the new format (e.g., dotnet package add). This change reflects the more consistent command ordering introduced in .NET 10 SDK.
Key changes:
- Command references updated from
dotnet add packagetodotnet package add - Command references updated from
dotnet remove packagetodotnet package remove - Command references updated from
dotnet list packagetodotnet package list
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| NuGet-Cross-Platform-Authentication-Plugin.md | Updates interactive command reference to use new syntax |
| dotnet-Commands.md | Updates command list table with new command syntax |
| NuGet-Client-SDK.md | Updates code examples to use new command syntax |
| install-and-use-a-package-using-the-dotnet-cli.md | Updates quickstart guide with new command syntax |
| Deprecate-packages.md | Updates deprecated package listing command syntax |
| install-use-packages-dotnet-cli.md | Comprehensive updates to all command references and examples |
| PackageDownload-Functionality.md | Updates table and tooling section with new syntax |
| Package-Source-Mapping.md | Updates onboarding instructions to use new command syntax |
| Package-Restore.md | Updates package addition command reference |
| Finding-and-Choosing-Packages.md | Updates prerelease package command reference |
| package-installation-process.md | Updates cache behavior description with new syntax |
| Security-Best-Practices.md | Updates dependency listing command references |
| Auditing-Packages.md | Updates vulnerability checking command references |
Comments suppressed due to low confidence (1)
docs/consume-packages/install-use-packages-dotnet-cli.md:1
- The link URL
/dotnet/core/tools/dotnet-list-packagepoints to the old command documentation, but the text describes the new commanddotnet package list. This URL should be updated to match the new command's documentation location.
---
docs/quickstart/install-and-use-a-package-using-the-dotnet-cli.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 45bf89e:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/concepts/Auditing-Packages.md | View | Details | |
| docs/concepts/Security-Best-Practices.md | 💡Suggestion | View | Details |
| docs/nuget-org/Deprecate-packages.md | 💡Suggestion | View | Details |
| docs/concepts/package-installation-process.md | ✅Succeeded | View | |
| docs/consume-packages/Finding-and-Choosing-Packages.md | ✅Succeeded | View | |
| docs/consume-packages/install-use-packages-dotnet-cli.md | ✅Succeeded | View | |
| docs/consume-packages/Package-Restore.md | ✅Succeeded | View | |
| docs/consume-packages/Package-Source-Mapping.md | ✅Succeeded | View | |
| docs/consume-packages/PackageDownload-Functionality.md | ✅Succeeded | View | |
| docs/quickstart/install-and-use-a-package-using-the-dotnet-cli.md | ✅Succeeded | View | |
| docs/reference/dotnet-Commands.md | ✅Succeeded | View | |
| docs/reference/extensibility/NuGet-Cross-Platform-Authentication-Plugin.md | ✅Succeeded | View | |
| docs/reference/NuGet-Client-SDK.md | ✅Succeeded | View |
docs/concepts/Auditing-Packages.md
- Line 23, Column 71: [Warning: bookmark-not-found - See documentation]
Cannot find bookmark '#dotnet-list-package---vulnerable' in 'concepts/Auditing-Packages.md'.
docs/concepts/Security-Best-Practices.md
- Line 2, Column 1: [Suggestion: ms-date-missing - See documentation]
Missing required attribute: 'ms.date'.
docs/nuget-org/Deprecate-packages.md
- Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation]
Invalid value for 'ms.author', 'anangaur' is not a valid Microsoft alias.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit b8f41c2:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/concepts/Auditing-Packages.md | View | Details | |
| docs/concepts/Security-Best-Practices.md | 💡Suggestion | View | Details |
| docs/nuget-org/Deprecate-packages.md | 💡Suggestion | View | Details |
| docs/concepts/package-installation-process.md | ✅Succeeded | View | |
| docs/consume-packages/Finding-and-Choosing-Packages.md | ✅Succeeded | View | |
| docs/consume-packages/install-use-packages-dotnet-cli.md | ✅Succeeded | View | |
| docs/consume-packages/Package-Restore.md | ✅Succeeded | View | |
| docs/consume-packages/Package-Source-Mapping.md | ✅Succeeded | View | |
| docs/consume-packages/PackageDownload-Functionality.md | ✅Succeeded | View | |
| docs/quickstart/install-and-use-a-package-using-the-dotnet-cli.md | ✅Succeeded | View | |
| docs/reference/dotnet-Commands.md | ✅Succeeded | View | |
| docs/reference/extensibility/NuGet-Cross-Platform-Authentication-Plugin.md | ✅Succeeded | View | |
| docs/reference/NuGet-Client-SDK.md | ✅Succeeded | View |
docs/concepts/Auditing-Packages.md
- Line 23, Column 71: [Warning: bookmark-not-found - See documentation]
Cannot find bookmark '#dotnet-list-package---vulnerable' in 'concepts/Auditing-Packages.md'.
docs/concepts/Security-Best-Practices.md
- Line 2, Column 1: [Suggestion: ms-date-missing - See documentation]
Missing required attribute: 'ms.date'.
docs/nuget-org/Deprecate-packages.md
- Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation]
Invalid value for 'ms.author', 'anangaur' is not a valid Microsoft alias.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
|
Learn Build status updates of commit 5825b45:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/concepts/Auditing-Packages.md | View | Details | |
| docs/concepts/Security-Best-Practices.md | 💡Suggestion | View | Details |
| docs/nuget-org/Deprecate-packages.md | 💡Suggestion | View | Details |
| docs/concepts/package-installation-process.md | ✅Succeeded | View | |
| docs/consume-packages/Finding-and-Choosing-Packages.md | ✅Succeeded | View | |
| docs/consume-packages/install-use-packages-dotnet-cli.md | ✅Succeeded | View | |
| docs/consume-packages/Package-Restore.md | ✅Succeeded | View | |
| docs/consume-packages/Package-Source-Mapping.md | ✅Succeeded | View | |
| docs/consume-packages/PackageDownload-Functionality.md | ✅Succeeded | View | |
| docs/quickstart/install-and-use-a-package-using-the-dotnet-cli.md | ✅Succeeded | View | |
| docs/reference/dotnet-Commands.md | ✅Succeeded | View | |
| docs/reference/extensibility/NuGet-Cross-Platform-Authentication-Plugin.md | ✅Succeeded | View | |
| docs/reference/NuGet-Client-SDK.md | ✅Succeeded | View |
docs/concepts/Auditing-Packages.md
- Line 23, Column 71: [Warning: bookmark-not-found - See documentation]
Cannot find bookmark '#dotnet-list-package---vulnerable' in 'concepts/Auditing-Packages.md'.
docs/concepts/Security-Best-Practices.md
- Line 2, Column 1: [Suggestion: ms-date-missing - See documentation]
Missing required attribute: 'ms.date'.
docs/nuget-org/Deprecate-packages.md
- Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation]
Invalid value for 'ms.author', 'anangaur' is not a valid Microsoft alias.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
donnie-msft
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good - I see several comments from Copilot and the warnings/suggestions from the Docs bot. Would be good to address those before merging.
Update our documentations to follow the latest dotnet sdk command names according to dotnet/sdk#45384