Skip to content

revert(release): remove the release workflow skill - #71

Merged
sentomk merged 1 commit into
mainfrom
revert-69-chore/release-skill
Oct 29, 2026
Merged

sentomk merged 1 commit into
mainfrom
revert-69-chore/release-skill

Conversation

@sentomk

@sentomk sentomk commented Oct 29, 2026 •

Copy link
Copy Markdown
Owner

Summary

Revert #69. Its squash commit on main has a stray w'w appended to the subject (chore(release): add a release workflow skill (#69)w'w), added when the squash message was edited. The skill content itself is unchanged and will be re-added in a new pull request with a clean message.

Changes

Testing

  • Docs-only revert; git diff against the parent of 244a113 is empty after this change.

@biobo-bot biobo-bot Bot left a comment

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.

Clean revert of #69 — approving.

Checked

  • The diff is exactly the inverse of #69: it deletes .claude/skills/release/SKILL.md (106 lines), the only file #69 added, and touches nothing else. No library code, tests, or docs are affected.
  • The branch is one commit ahead of main and zero behind, so it applies cleanly.
  • scripts/check_commit_msg.py explicitly allows Revert "..." subjects, so the title passes the commit-message rule.

Two small notes (non-blocking)

  • The commit subject ends in a stray w'w (... (#69)w'w"). Squash-merge uses the PR title, which is clean, so this won't land on main unless the commit is merged as-is.
  • The PR body only says Reverts sentomk/patternia#69. If validate.yml requires the bold Summary / Changes / Testing sections on every PR, that check may flag it; a one-line reason for the revert would also help future readers.

@codecov

codecov Bot commented Oct 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sentomk sentomk changed the title Revert "chore(release): add a release workflow skill" revert(release): remove the release workflow skill Oct 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Benchmark comparison (informational)

Base 244a113 vs PR 2db2385, run one after the other on the same GitHub-hosted runner. This does not block merging.

Noise estimate: the 20 control rows below do not run Patternia code, so they should not change between base and PR. As a group they moved by a median of +0.0% (run order and machine drift); that offset is divided out of the Patternia deltas (Adjusted). The controls then scatter by a median of 0.2%. Adjusted differences under 5.0% are treated as noise.

Patternia: 8 benchmarks, 0 slower, 0 faster, 8 within noise.

Benchmark Base (ns) PR (ns) Delta Adjusted Verdict
PatterniaPipe CommandParser 1.775 1.774 -0.1% -0.1% within noise
PatterniaPipe PacketMixed 1.621 1.623 +0.1% +0.1% within noise
PatterniaPipe PacketMixedHeavyBind 1.685 1.686 +0.1% +0.0% within noise
PatterniaPipe ProtocolRouter 1.633 1.627 -0.4% -0.4% within noise
PatterniaPipe VariantAltHot 1.559 1.558 -0.1% -0.1% within noise
PatterniaPipe VariantAltHotGuarded 1.715 1.714 -0.1% -0.1% within noise
PatterniaPipe VariantMixed 1.998 2.000 +0.1% +0.0% within noise
PatterniaPipe VariantMixedGuarded 2.012 1.996 -0.8% -0.8% within noise
Control rows (not Patternia code)
Benchmark Base (ns) PR (ns) Delta
IfElse CommandParser 2.205 2.208 +0.1%
IfElse ProtocolRouter 1.868 1.866 -0.1%
Sequential VariantAltHot 1.714 1.715 +0.1%
Sequential VariantAltHotGuarded 1.870 1.878 +0.4%
Sequential VariantMixed 1.164 1.164 +0.0%
Sequential VariantMixedGuarded 1.432 1.433 +0.1%
StdVisit CommandParser 1.389 1.386 -0.2%
StdVisit ProtocolRouter 1.520 1.518 -0.2%
StdVisit VariantAltHot 1.558 1.559 +0.1%
StdVisit VariantAltHotGuarded 1.716 1.713 -0.2%
StdVisit VariantMixed 1.992 1.993 +0.1%
StdVisit VariantMixedGuarded 2.154 2.152 -0.1%
SwitchIndex VariantAltHot 1.715 1.714 -0.1%
SwitchIndex VariantAltHotGuarded 1.870 1.871 +0.1%
SwitchIndex VariantMixed 0.899 0.905 +0.7%
SwitchIndex VariantMixedGuarded 1.402 1.396 -0.4%
Switch CommandParser 2.253 2.256 +0.1%
Switch PacketMixed 1.439 1.430 -0.6%
Switch PacketMixedHeavyBind 1.501 1.502 +0.1%
Switch ProtocolRouter 1.595 1.592 -0.2%

Raw results and the comparison chart are attached to this workflow run as an artifact. Shared runners are noisy, so use this as a trend, not a measurement.

@sentomk
sentomk merged commit 6ba27eb into main Oct 29, 2026
24 of 25 checks passed
@sentomk
sentomk deleted the revert-69-chore/release-skill branch October 29, 2026 16:45
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.

1 participant