Skip to content

fix: remove auto-publish, make releases explicit#57

Merged
dean0x merged 2 commits intomainfrom
fix/remove-dead-release-workflow
Feb 21, 2026
Merged

fix: remove auto-publish, make releases explicit#57
dean0x merged 2 commits intomainfrom
fix/remove-dead-release-workflow

Conversation

@dean0x
Copy link
Owner

@dean0x dean0x commented Feb 21, 2026

Summary

  • Deletes the broken .github/workflows/release.yml (triggered on GitHub release events, always failed at npm test)
  • Removes the auto-publish release job from ci.yml — merging to main no longer triggers npm publish
  • Adds new release.yml with workflow_dispatch trigger — releases are now explicitly requested from the GitHub Actions UI
  • Updates CLAUDE.md release process docs

Closes #50

What changed

Before After
ci.yml auto-published on merge to main when version changed ci.yml only runs tests
Broken release.yml triggered on GitHub release events New release.yml triggered manually via workflow_dispatch
Two competing release mechanisms (both broken in different ways) One explicit release mechanism

Test plan

  • PR CI passes (test job only)
  • After merge: release.yml appears in Actions tab with "Run workflow" button
  • Manual trigger: validates version, checks release notes, runs tests, publishes

release.yml triggered on GitHub release publish events but always
failed at `npm test` (which exits 1 by design). ci.yml already
handles publishing on push to main. Removes the broken duplicate.
@qodo-free-for-open-source-projects

Review Summary by Qodo

Remove dead release.yml workflow

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Removes broken .github/workflows/release.yml workflow
• Workflow triggered on GitHub releases but always failed at npm testci.yml already handles publishing on push to main
• Resolves duplicate and non-functional release mechanism
Diagram
flowchart LR
  A["release.yml<br/>GitHub Release Trigger"] -->|"Deleted"| B["Removed<br/>Broken Workflow"]
  C["ci.yml<br/>Push to Main"] -->|"Remains Active"| D["Sole Publishing<br/>Mechanism"]
Loading

Grey Divider

File Changes

1. .github/workflows/release.yml 🐞 Bug fix +0/-55

Delete non-functional release workflow file

• Entire workflow file deleted (55 lines removed)
• Workflow triggered on GitHub release publish events
• Contained npm test step that always failed (exits 1 by design)
• Duplicate of publishing logic already in ci.yml

.github/workflows/release.yml


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

- Remove auto-publish release job from ci.yml
- Add new release.yml triggered manually from GitHub Actions UI
- Merging a version bump to main no longer auto-publishes
- Update CLAUDE.md release process docs
@dean0x dean0x changed the title fix: remove dead release.yml workflow fix: remove auto-publish, make releases explicit Feb 21, 2026
@dean0x dean0x merged commit e149b7e into main Feb 21, 2026
1 check passed
@dean0x dean0x deleted the fix/remove-dead-release-workflow branch February 21, 2026 09:36
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.

fix: Remove dead release.yml workflow

1 participant