Skip to content

ci: remove release-please, use manual releases#4

Open
Eric-Stauffer wants to merge 5 commits intomainfrom
remove-release-please
Open

ci: remove release-please, use manual releases#4
Eric-Stauffer wants to merge 5 commits intomainfrom
remove-release-please

Conversation

@Eric-Stauffer
Copy link
Copy Markdown

Summary

  • Removes release-please (config, manifest, workflow)
  • Releases are now created manually via GitHub UI
  • The publish-gpr job in ci.yml triggers automatically on release: types: [created]

How to publish

  1. Go to Releases > "Create a new release"
  2. Create a tag (e.g. v1.0.0)
  3. Publish — the workflow handles the rest

🤖 Generated with Claude Code

Eric-Stauffer and others added 4 commits April 15, 2026 12:58
Releases will be created manually through the GitHub UI.
The publish-gpr job in ci.yml triggers on release creation.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Prevents noisy failures on PRs and pushes when the version
already exists on GitHub Packages.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Comment thread .github/workflows/ci.yml Outdated
node-version: "22.14.0"
registry-url: https://npm.pkg.github.com/

- run: npm install
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Why did we add this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This was copied from the remi-marketing-analytics workflow to match the pattern exactly. But it's not actually needed here -- the publish step only needs the package files (which are raw TypeScript in src/), not node_modules. We can remove it if you'd prefer.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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