Skip to content

Add Claude skills to help release the package#229

Merged
StephenWakely merged 4 commits intomasterfrom
stephen/skills
Apr 28, 2026
Merged

Add Claude skills to help release the package#229
StephenWakely merged 4 commits intomasterfrom
stephen/skills

Conversation

@StephenWakely
Copy link
Copy Markdown
Contributor

Largely following the release process outlined here add a couple of skills.

  1. prepare the release (Release 9.1.0 #228 is the first example of this)
  2. push the new release to nuget

@StephenWakely StephenWakely requested a review from a team as a code owner April 27, 2026 11:30
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9bd3f908ba

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .claude/skills/publish-release/SKILL.md Outdated
git pull origin master
```

Verify the top commit is the release commit (message should be "Release version X.Y.Z"). If it is not, stop and tell the user to ensure the release PR is merged before running this skill.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Don't require the release commit at HEAD

When the release PR is merged with GitHub's normal merge/squash flow, HEAD after git pull is the merge/squash commit, not necessarily the branch commit created in Step 5. This repo's history already has release/version changes landing through merge commits, and this skill creates the PR title as Release X.Y.Z, so a successfully merged release can fail this exact Release version X.Y.Z check and stop before tagging/publishing. Verify the csproj version or that the release commit is contained in the merge instead of requiring the exact HEAD subject.

Useful? React with 👍 / 👎.

Comment thread .claude/skills/prepare-release/SKILL.md Outdated

## Step 1: Confirm Version

If the user did not supply a version number, read `src/StatsdClient/StatsdClient.csproj` and show the current `<PackageVersion>`. Ask: "Prepare release for X.Y.Z?" and wait for confirmation before proceeding.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Fall back to <Version> when confirming the release

In the current src/StatsdClient/StatsdClient.csproj, there is no <PackageVersion> element; the package version currently comes from <Version>9.0.0</Version>. When a user invokes prepare-release without explicitly supplying a version, this instruction has no value to read or show for confirmation, so the skill cannot determine the current release version before creating the branch. Read <PackageVersion> if present and otherwise fall back to <Version>.

Useful? React with 👍 / 👎.

Comment thread .claude/skills/publish-release/SKILL.md Outdated
## Step 3: Tag the Release

```bash
git tag X.Y.Z
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.

Should we use git directly or instead use gh release create instead? See: https://cli.github.com/manual/gh_release_create

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good call.

@StephenWakely StephenWakely merged commit cc8fe3b into master Apr 28, 2026
27 checks passed
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.

2 participants