Skip to content

ci: remove manual publish.yml, rely on release-please.yml#754

Merged
calclavia merged 1 commit into
mainfrom
calclavia/rm-publish-yml
May 1, 2026
Merged

ci: remove manual publish.yml, rely on release-please.yml#754
calclavia merged 1 commit into
mainfrom
calclavia/rm-publish-yml

Conversation

@calclavia
Copy link
Copy Markdown
Contributor

Summary

Removes the manual Publish NPM workflow. release-please.yml's embedded publish job is the canonical path now that OIDC trust on npmjs.com is set for release-please.yml.

Why

  • npm Trusted Publishers can target only one workflow file at a time per package
  • We picked release-please.yml so every release PR merge auto-publishes with provenance
  • publish.yml was the manual fallback we used to ship smithery@1.0.0 — no longer needed

🤖 Generated with Claude Code

Consolidates to a single publish path. release-please.yml's publish
job auto-runs on release PR merge with provenance; publish.yml was a
manual fallback that's no longer needed now that OIDC trust is set up
for release-please.yml.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@calclavia calclavia enabled auto-merge (squash) May 1, 2026 08:01
@calclavia calclavia merged commit faa8f7e into main May 1, 2026
4 checks passed
@calclavia calclavia deleted the calclavia/rm-publish-yml branch May 1, 2026 08:01
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