Skip to content
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

Add docs for the 'dotnet nuget why' command #41472

Merged
merged 6 commits into from
Jun 19, 2024
Merged

Conversation

advay26
Copy link
Member

@advay26 advay26 commented Jun 19, 2024

Summary

Fixes: NuGet/Home#13499

Creating docs for a new dotnet CLI command that we added recently: dotnet nuget why.
Design spec: dotnet nuget why command
Implementation: NuGet/NuGet.Client#5761


Internal previews

📄 File 🔗 Preview link
docs/core/tools/dotnet-nuget-why.md dotnet nuget why
docs/core/tools/index.md .NET CLI

docs/core/tools/dotnet-nuget-why.md Outdated Show resolved Hide resolved
Copy link
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

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

The new article needs to be added to the toc.yml file at https://github.com/dotnet/docs/blob/main/docs/navigate/tools-diagnostics/toc.yml

docs/core/tools/dotnet-nuget-why.md Outdated Show resolved Hide resolved
@advay26 advay26 requested a review from a team as a code owner June 19, 2024 22:51
@advay26 advay26 requested a review from tdykstra June 19, 2024 22:51
@tdykstra tdykstra merged commit c1bf9bf into main Jun 19, 2024
8 checks passed
@tdykstra tdykstra deleted the dev-advay26-dotnet-nuget-why branch June 19, 2024 23:16
---
# dotnet nuget why

**This article applies to:** ✔️ .NET 8.0.4xx SDK and later versions
Copy link
Member

Choose a reason for hiding this comment

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

@advay26 This version looks wrong, I think

Copy link
Member

Choose a reason for hiding this comment

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

@Youssef1313 We have a bug where it's not showing up in dotnet nuget --help, because of an incomplete transition to a different CLI parsing library:

but if you run dotnet nuget why anyway, you'll see why's help (after error messages talking about missing required arguments)

Otherwise, please explain why you think it's the wrong version.

Copy link
Member

Choose a reason for hiding this comment

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

It didn't work for me with .NET 9 SDK Preview 4 but worked with preview 6.

Copy link
Member

@zivkan zivkan Jul 12, 2024

Choose a reason for hiding this comment

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

.NET 9 P4 came out in May, whereas 8.0.400 and .NET 9 P6 came out in July. 8.0.400 isn't out yet. Basically, it's a new feature.

I'll leave it up to the .NET team to decide if they want to update the docs to mention which pre-release version of .NET 9 the feature is available. My opinion is that once .NET 9 goes GA, it won't be relevant any longer and the NuGet team at least doesn't have dedicated docs writer resources to help us keep docs up to date.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for the clarification. I agree the exact preview isn't very relevant.

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.

Add docs for dotnet nuget why
5 participants