Skip to content

do not use changesets to do the publish, publish manually instead#28

Merged
jkadamczyk merged 1 commit intomasterfrom
kuba/fix-publishing
Jan 26, 2026
Merged

do not use changesets to do the publish, publish manually instead#28
jkadamczyk merged 1 commit intomasterfrom
kuba/fix-publishing

Conversation

@jkadamczyk
Copy link
Collaborator

@jkadamczyk jkadamczyk commented Jan 26, 2026

Summary by CodeRabbit

  • Chores
    • Updated the release and publishing workflow to separate Release PR creation from npm publishing, implementing OIDC Trusted Publishing for enhanced security and reliability.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 26, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

GitHub Actions release workflow refactored to decouple Release PR creation from npm publishing. The publish step now runs separately as a conditional step using OIDC Trusted Publishing when changesets are absent.

Changes

Cohort / File(s) Summary
Release Workflow Refactoring
.github/workflows/release.yml
Split combined Release PR/publish step into two separate steps: (1) Create Release Pull Request step simplified to remove direct publish invocation and environment variable configuration, (2) New Publish to npm step added with OIDC Trusted Publishing, conditioned on steps.changesets.outputs.hasChangesets == 'false'. Removed NODE_AUTH_TOKEN and NPM_CONFIG_PROVENANCE environment variables from Release PR step.

Comment @coderabbitai help to get the list of available commands and usage tips.

@jkadamczyk jkadamczyk merged commit 0fc70df into master Jan 26, 2026
18 of 19 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