Introduce parent "Sources" section & add Audit Source options#3530
Introduce parent "Sources" section & add Audit Source options#3530donnie-msft merged 8 commits intomainfrom
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
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 |
nkolev92
left a comment
There was a problem hiding this comment.
Minor comments.
Screenshots look great.
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 a9298e8: ✅ Validation status: passed
For more details, please refer to the build report. |
There was a problem hiding this comment.
Pull request overview
This pull request introduces a parent "Sources" section in the NuGet Visual Studio Options documentation and adds comprehensive documentation for Audit Source options. The changes align with GitHub issue #14687.
Changes:
- Generalized "Package Sources" page to "Sources" to accommodate both package sources and audit sources
- Added detailed documentation for the new Audit Sources feature, including configuration steps and three supporting screenshots
- Updated existing screenshots to reflect the new UI structure
- Fixed minor punctuation and formatting issues for consistency
Reviewed changes
Copilot reviewed 2 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| docs/consume-packages/nuget-visual-studio-options.md | Renamed "Package Sources" section to "Sources", added "Managing Audit Sources" subsection with detailed configuration instructions, updated image references, and fixed minor punctuation issues |
| docs/consume-packages/media/vsoptions/audit-sources-1.png | New screenshot showing the audit sources checkbox option |
| docs/consume-packages/media/vsoptions/audit-sources-2.png | New screenshot showing empty audit sources table |
| docs/consume-packages/media/vsoptions/audit-sources-3.png | New screenshot showing configured audit source for nuget.org |
| docs/consume-packages/media/vsoptions/package-source-machine-wide.png | Updated screenshot to reflect new UI |
| docs/consume-packages/media/vsoptions/sources-page.png | New screenshot replacing package-sources-page.png with updated UI |
| docs/consume-packages/install-use-packages-visual-studio.md | Updated anchor link from #package-sources to #sources |
Comments suppressed due to low confidence (1)
docs/consume-packages/nuget-visual-studio-options.md:100
- According to the .NET Documentation Guidelines (CodingGuidelineID: 1000000), ordered and unordered lists should use complete sentences with proper punctuation, ending with a period if it's more than three words. The list items here exceed three words but don't end with periods. Consider adding periods to maintain consistency with the documentation standards.
- **Name**: Display name for the source
- **Source**: URL or file path for the package source
- **Enabled checkbox**: Enable or disable a source without removing it.
- **Allow Insecure Connections**: Enable or disable allowing insecure HTTP connections
Fixes: NuGet/Home#14687
"Package Sources" page is now generalized as "Sources".