Skip to content

Add note for .NET tool dependencies (#10020)#10736

Open
abdulrahmanhossam wants to merge 2 commits intoNuGet:devfrom
abdulrahmanhossam:fix-dotnet-tool-dependencies-note
Open

Add note for .NET tool dependencies (#10020)#10736
abdulrahmanhossam wants to merge 2 commits intoNuGet:devfrom
abdulrahmanhossam:fix-dotnet-tool-dependencies-note

Conversation

@abdulrahmanhossam
Copy link
Copy Markdown

Fixes #10020

Description
Updated the DependencySetsViewModel and DisplayPackageViewModelFactory to pass the IsDotnetTool flag.
Modified _PackageDependencies.cshtml to display a note with a link to issue #10020 when a package is a .NET Tool and has no explicit dependencies in its nuspec.

Expected Behavior
Instead of showing "This package has no dependencies.", .NET tools will now show: ".NET tools have dependencies embedded in the package." along with a link to gather upvotes for improving the experience.

@abdulrahmanhossam abdulrahmanhossam requested a review from a team as a code owner March 11, 2026 13:47
@abdulrahmanhossam abdulrahmanhossam changed the base branch from main to dev March 11, 2026 13:53
@abdulrahmanhossam
Copy link
Copy Markdown
Author

Hi @joelverhagen, thanks for the assignment! 🚀

I've updated the base branch to dev as requested.

Here is a mockup screenshot of how the UI will look with the proposed changes:
b9c07260-010f-485c-b91a-a2baa55c816f

I'm totally open to any feedback on the wording or implementation from the team! Let me know what you think.

@pranathibora14
Copy link
Copy Markdown
Contributor

Thank you for the contribution @abdulrahmanhossam! 🎉

One small suggestion on the wording, instead of "See this issue to vote and improve this experience", how about:

.NET tools have dependencies embedded in the package. Join the discussion on improving this experience.

This sets the right expectation for users clicking through.

@Frulfump
Copy link
Copy Markdown

Makes sense (still funny that I'm the only one who has upvoted the issue in two years 😅)

@abdulrahmanhossam
Copy link
Copy Markdown
Author

Thanks for the feedback, @pranathibora14! That wording definitely makes more sense.

I've just pushed a commit to update the message as suggested. Let me know if there's anything else needed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NuGet.org Bug]: Empty dependencies list for .NET tools

3 participants