Skip to content

Update SKILL.md: file naming convention and PR description requirements#37153

Merged
tdykstra merged 2 commits into
mainfrom
copilot/update-whats-new-skill-file
May 15, 2026
Merged

Update SKILL.md: file naming convention and PR description requirements#37153
tdykstra merged 2 commits into
mainfrom
copilot/update-whats-new-skill-file

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 14, 2026

Fixes #37152

Two corrections to the What's New include content rules skill file.

File naming convention

  • Require whole words separated by hyphens — no abbreviations, no combining words with numbers
  • Changed preview suffix from -preview{N} to -preview-{N}
  • Updated all examples and validation checklist accordingly
Wrong:  infer-pk-display-name-preview2.md
Right:  infer-passkey-display-name-preview-2.md

PR description requirements

  • Added new section requiring Fixes dotnet/AspNetCore.Docs#{ISSUE NUMBER} as the first line of every PR description, followed by a blank line

Internal previews

📄 File 🔗 Preview link
.github/skills/whats-new-include-content-rules/SKILL.md .github/skills/whats-new-include-content-rules/SKILL

Copilot AI changed the title [WIP] Update SKILL.md for Whats New generation Update SKILL.md: file naming convention and PR description requirements May 14, 2026
Copilot AI requested a review from wadepickett May 14, 2026 17:34
@wadepickett wadepickett requested a review from tdykstra May 14, 2026 17:46
@wadepickett wadepickett marked this pull request as ready for review May 14, 2026 17:48

* Lowercase, hyphenated, descriptive names.
* Append the preview number suffix to each filename: `-preview{N}` (where `{N}` is the preview number, e.g., `-preview2`).
* Lowercase, hyphenated, descriptive names using whole words only.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

How about an exception for very long words


* The very first line of the PR description must be `Fixes dotnet/AspNetCore.Docs#{ISSUE NUMBER}` (where `{ISSUE NUMBER}` is the number of the issue this PR addresses), followed by a blank line before any other content.
* Example:
```
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If multiple issues then multiple lines, or some other way to handle them

@tdykstra tdykstra merged commit 6d0c755 into main May 15, 2026
4 checks passed
@tdykstra tdykstra deleted the copilot/update-whats-new-skill-file branch May 15, 2026 01:02
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.

Update Agent and skill for What's New generation: filename & issue number instructions

3 participants