Skip to content

Add Package name limits section to publish-a-package#3574

Open
seaniyer wants to merge 8 commits into
mainfrom
seiyer-package-name-update
Open

Add Package name limits section to publish-a-package#3574
seaniyer wants to merge 8 commits into
mainfrom
seiyer-package-name-update

Conversation

@seaniyer
Copy link
Copy Markdown
Contributor

@seaniyer seaniyer commented May 26, 2026

Adds a new Package name limits section above Package size limits in docs/nuget-org/Publish-a-package.md.

The section briefly sets context and lists the four naming rules enforced by nuget.org:

Copilot AI review requested due to automatic review settings May 26, 2026 19:45
@seaniyer seaniyer requested review from a team as code owners May 26, 2026 19:45
@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 94d4c5a:

✅ Validation status: passed

File Status Preview URL Details
docs/nuget-org/Publish-a-package.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

Copy link
Copy Markdown
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 a new Package name limits section to Publish-a-package.md to document nuget.org’s package ID naming constraints and the rationale for enforcing them (preventing normalization collisions that can affect URLs and file paths).

Changes:

  • Added a Package name limits section above Package size limits.
  • Documented a short rationale for package ID naming restrictions.
  • Listed four nuget.org-enforced package naming rules.

Comment thread docs/nuget-org/Publish-a-package.md Outdated
Comment thread docs/nuget-org/Publish-a-package.md Outdated
Comment thread docs/nuget-org/Publish-a-package.md Outdated
Comment thread docs/nuget-org/Publish-a-package.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 3226031:

✅ Validation status: passed

File Status Preview URL Details
docs/nuget-org/Publish-a-package.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit b316fca:

💡 Validation status: suggestions

File Status Preview URL Details
docs/reference/errors-and-warnings/NU1017.md 💡Suggestion View Details
docs/nuget-org/Publish-a-package.md ✅Succeeded View
docs/reference/nuspec.md ✅Succeeded View

docs/reference/errors-and-warnings/NU1017.md

  • Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'Nigusu-Allehu' is not a valid Microsoft alias.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 9a89332:

✅ Validation status: passed

File Status Preview URL Details
docs/nuget-org/Publish-a-package.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit b42cc74:

✅ Validation status: passed

File Status Preview URL Details
docs/nuget-org/Publish-a-package.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 7d34a84:

✅ Validation status: passed

File Status Preview URL Details
docs/nuget-org/Publish-a-package.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

Comment thread docs/nuget-org/Publish-a-package.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 6b492af:

✅ Validation status: passed

File Status Preview URL Details
docs/nuget-org/Publish-a-package.md ✅Succeeded View
docs/reference/nuspec.md ✅Succeeded View

For more details, please refer to the build report.

Copy link
Copy Markdown
Contributor Author

@seaniyer seaniyer left a comment

Choose a reason for hiding this comment

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

Thanks, Copilot detected the wrong char count also exists in https://github.com/NuGet/docs.microsoft.com-nuget/blob/main/docs/reference/nuspec.md
So I'm fixing that as well

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

nkolev92
nkolev92 previously approved these changes May 26, 2026
@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 77df6f3:

✅ Validation status: passed

File Status Preview URL Details
docs/nuget-org/Publish-a-package.md ✅Succeeded View
docs/reference/nuspec.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

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.

4 participants