Skip to content

Move breaking-change-doc scripts into scripts/ and use bare skill-relative paths #127977

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
lewing wants to merge 2 commits into
base: main
Choose a base branch
from
+6 −6
Open

Move breaking-change-doc scripts into scripts/ and use bare skill-relative paths #127977

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
24b0d8c
Move breaking-change-doc scripts into scripts/ subdirectory
lewing May 8, 2026
6324891
Use bare skill-relative script paths to match the spec example
lewing May 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .github/skills/breaking-change-doc/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ migration guidance. This skill automates that process:
| `.github/workflows/breaking-change-doc.md` | gh-aw workflow — triggers on PR merge/label |
| `.github/workflows/breaking-change-doc.lock.yml` | Compiled workflow (generated by `gh aw compile`) |
| `.github/skills/breaking-change-doc/SKILL.md` | This skill |
| `.github/skills/breaking-change-doc/Get-VersionInfo.ps1` | `gh` CLI-based .NET version detection |
| `.github/skills/breaking-change-doc/Build-IssueComment.ps1` | Builds PR comment with URL-encoded issue creation link |
| `.github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1` | `gh` CLI-based .NET version detection |
| `.github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1` | Builds PR comment with URL-encoded issue creation link |

---

Expand Down Expand Up @@ -102,7 +102,7 @@ include the actual area label in the description.
Run the helper script to determine the .NET version context:

```
pwsh .github/skills/breaking-change-doc/Get-VersionInfo.ps1 -PrNumber <number>
pwsh scripts/Get-VersionInfo.ps1 -PrNumber <number>
```

**You MUST display the complete script output.** The script outputs JSON that includes:
Expand Down Expand Up @@ -239,7 +239,7 @@ Pass the PR assignees so they are `@`-mentioned in the comment and receive a
GitHub notification:

```bash
pwsh .github/skills/breaking-change-doc/Build-IssueComment.ps1 \
pwsh scripts/Build-IssueComment.ps1 \
-IssueDraftPath artifacts/docs/breakingChanges/issue-draft.md \
-Title "<the [Breaking change]: ... title from Step 5>" \
-Assignees "@user1 @user2" \
Expand Down
2 changes: 1 addition & 1 deletion ...reaking-change-doc/Build-IssueComment.ps1 → ...change-doc/scripts/Build-IssueComment.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
# - An email reminder
#
# Usage:
# pwsh .github/skills/breaking-change-doc/Build-IssueComment.ps1 `
# pwsh .github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1 `
# -IssueDraftPath issue-draft.md `
# -Title "[Breaking change]: Something changed" `
# -Assignees "@user1 @user2" `
Expand Down
2 changes: 1 addition & 1 deletion ...s/breaking-change-doc/Get-VersionInfo.ps1 → ...ng-change-doc/scripts/Get-VersionInfo.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# Determines the .NET version context for a merged PR using the GitHub CLI (gh).
#
# Usage:
# pwsh .github/skills/breaking-change-doc/Get-VersionInfo.ps1 -PrNumber 114929
# pwsh .github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1 -PrNumber 114929
#
# Output: JSON object with LastTagBeforeMerge, FirstTagWithChange, EstimatedVersion

Expand Down