Skip to content

Commit 39a08fd

Browse files
nkolev92Copilot
andcommitted
RemoteSourceDependencyInfo: add protocol context to NULL_INC remark
ContentUri maps to the required packageContent field in the NuGet V3 registration API, so null is not expected in practice. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent b16af0b commit 39a08fd

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/NuGet.Core/NuGet.Protocol/RemoteSourceDependencyInfo.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,9 @@ public RemoteSourceDependencyInfo(
6565
/// <remarks>
6666
/// NULL_INC: Annotated as non-null but no runtime check is enforced in the constructor
6767
/// to avoid introducing a new throw in a previously-permissive code path.
68-
/// Revisit with telemetry to confirm callers never pass null.
68+
/// This maps to the <c>packageContent</c> field in the NuGet V3 registration API
69+
/// (https://learn.microsoft.com/en-us/nuget/api/registration-base-url-resource#registration-leaf-object-in-a-page)
70+
/// which is required by protocol, so null is not expected in practice.
6971
/// </remarks>
7072
public string ContentUri { get; set; }
7173

0 commit comments

Comments
 (0)