Skip to content

Add a copilot icon to the .NET nuget package#688

Merged
stephentoub merged 1 commit intomainfrom
stoub/packageicon
Mar 10, 2026
Merged

Add a copilot icon to the .NET nuget package#688
stephentoub merged 1 commit intomainfrom
stoub/packageicon

Conversation

@stephentoub
Copy link
Collaborator

The icon shows up in nuget when browsing/searching packages.

Before:
image

After:
image

I've used https://github.com/microsoft/vscode-copilot-chat/blob/main/assets/copilot.png. If there's a better one, we can switch to that instead.

@stephentoub stephentoub requested a review from a team as a code owner March 5, 2026 15:59
Copilot AI review requested due to automatic review settings March 5, 2026 15:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a NuGet package icon for the .NET GitHub.Copilot.SDK package so the Copilot icon appears in NuGet browse/search results.

Changes:

  • Set PackageIcon in GitHub.Copilot.SDK.csproj.
  • Pack assets/copilot.png into the NuGet package root so NuGet can display it.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
dotnet/src/GitHub.Copilot.SDK.csproj Configures NuGet packing to include and reference the new package icon.
assets/copilot.png Adds the icon image that will be embedded into the NuGet package.

@stephentoub stephentoub enabled auto-merge March 5, 2026 16:49
@github-actions
Copy link
Contributor

Cross-SDK Consistency Review ✅

This PR adds a package icon to the .NET NuGet package, which improves discoverability when browsing/searching packages on nuget.org.

Consistency Analysis

I've reviewed whether similar package icon metadata should be added to the other SDK implementations:

✅ .NET (this PR)

  • Status: Icon added via (PackageIcon)copilot.png(/PackageIcon) in the .csproj
  • Ecosystem support: NuGet fully supports package icons and displays them prominently in search results and package pages

🟡 Node.js/TypeScript

  • Status: No icon metadata in package.json
  • Ecosystem support: npm does NOT support package icons in package metadata. npm displays icons from the GitHub repository (via social preview images) on the package page, but there's no icon field in package.json
  • Recommendation: No action needed - npm doesn't support this feature

🟡 Python

  • Status: No icon metadata in pyproject.toml
  • Ecosystem support: PyPI does NOT support package icons in the traditional sense. Package icons/logos are displayed via the project's GitHub repository social preview or by embedding images in the README/long_description
  • Recommendation: No action needed - PyPI doesn't support package icon metadata

🟡 Go

  • Status: No icon metadata in go.mod
  • Ecosystem support: Go modules (pkg.go.dev) do NOT support package icon metadata. The Go package registry displays repository information but doesn't have a concept of package icons
  • Recommendation: No action needed - Go modules don't support this feature

Conclusion

This PR maintains appropriate cross-SDK consistency. The .NET SDK is correctly adopting a NuGet-specific feature that has no equivalent in the npm, PyPI, or Go module ecosystems. The shared assets/copilot.png icon can potentially be reused by documentation or other contexts across all SDKs.

No further changes needed

Generated by SDK Consistency Review Agent for issue #688 ·

@stephentoub stephentoub added this pull request to the merge queue Mar 10, 2026
Merged via the queue into main with commit abfbc9e Mar 10, 2026
26 checks passed
@stephentoub stephentoub deleted the stoub/packageicon branch March 10, 2026 10:55
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.

3 participants