Skip to content

Update dotnet-new-list.md with warning - #56208

Open
marcpopMSFT wants to merge 4 commits into
mainfrom
marcpopMSFT-patch-1
Open

marcpopMSFT wants to merge 4 commits into
mainfrom
marcpopMSFT-patch-1

Conversation

@marcpopMSFT

@marcpopMSFT marcpopMSFT commented Sep 22, 2026 •

Copy link
Copy Markdown
Member

Added warnings about template listing and directory trust.

Summary

We implicitly load a project from the current or above directory to limit the templates we show so we want to document that.

Fixes #Issue_Number (if available)


Internal previews

File Preview link
docs/core/tools/dotnet-new-list.md Learn preview

Build report

Added warnings about template listing and directory trust.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The warning must cover --project and be scoped to .NET SDK 7.0.100+.

Review effort: Lite
Findings: None

What changed in this PR

Documents project discovery and directory trust for template listing.

Changes:

  • Adds a warning about inspecting projects in current or parent directories.
  • Advises using trusted directories.
File Description
docs/​core/​tools/​dotnet-new-list.md Adds the project-discovery warning; it needs --project coverage and an SDK version qualifier.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/core/tools/dotnet-new-list.md Outdated
Removed outdated author information and updated the date.
@gewarren

Copy link
Copy Markdown
Collaborator

Sorry, I didn't mean to make that commit in your branch. I'll revert it.

gewarren and others added 2 commits September 22, 2026 18:39
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants