Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 22, 2026

Description

The Microsoft.NET.ILLink and Microsoft.NET.ILLink.Tasks NuGet packages are published but lack documentation indicating they're SDK-internal and unsupported for direct consumption.

Changes

  • Added PACKAGE.md files to both packages with clear "Do not directly reference this package" language matching other SDK-internal packages like Microsoft.NETCore.App
  • Enabled package readme inclusion by removing EnableDefaultPackageReadmeFile=false from ILLink.Tasks.csproj and adding EnableDefaultPackageReadmeFile=true to Mono.Linker.csproj
  • Included trimming documentation links for users who need guidance on proper trimming usage through the SDK

Both readmes follow the standard repository PACKAGE.md structure with About, How to Use, Additional Documentation, and Feedback sections.

Original prompt

This section details on the original issue you should resolve

<issue_title>Illink nuget packages should state support contract in package readme</issue_title>
<issue_description>The illink packages aren’t supported outside of use by the SDK. We should state that in the readme. </issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: agocke <515774+agocke@users.noreply.github.com>
@dotnet-policy-service dotnet-policy-service bot added the linkable-framework Issues associated with delivering a linker friendly framework label Jan 22, 2026
Copilot AI changed the title [WIP] Update Illink NuGet packages readme to state support contract Add package readmes to illink NuGet packages stating SDK-only support Jan 22, 2026
Copilot AI requested a review from agocke January 22, 2026 18:11
@agocke agocke marked this pull request as ready for review January 22, 2026 19:12
Copilot AI review requested due to automatic review settings January 22, 2026 19:12
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

This PR adds package readme files to the Microsoft.NET.ILLink and Microsoft.NET.ILLink.Tasks NuGet packages to document that these packages are SDK-internal and should not be directly referenced by users.

Changes:

  • Added PACKAGE.md files to both illink packages with "Do not directly reference" warnings and trimming documentation links
  • Enabled package readme inclusion in project files by adding/removing EnableDefaultPackageReadmeFile properties

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/tools/illink/src/linker/PACKAGE.md New package readme for Microsoft.NET.ILLink stating SDK-only usage and providing trimming documentation links
src/tools/illink/src/linker/Mono.Linker.csproj Added EnableDefaultPackageReadmeFile=true to include the PACKAGE.md file in the NuGet package
src/tools/illink/src/ILLink.Tasks/PACKAGE.md New package readme for Microsoft.NET.ILLink.Tasks stating SDK-only usage and providing trimming documentation links
src/tools/illink/src/ILLink.Tasks/ILLink.Tasks.csproj Removed TODO comment and EnableDefaultPackageReadmeFile=false to enable package readme inclusion

@agocke agocke merged commit 0b9b34d into main Jan 23, 2026
98 checks passed
@github-project-automation github-project-automation bot moved this to Done in AppModel Jan 23, 2026
@agocke agocke deleted the copilot/update-readme-for-illink-packages branch January 23, 2026 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

linkable-framework Issues associated with delivering a linker friendly framework

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Illink nuget packages should state support contract in package readme

3 participants