Skip to content

ci: pass npm token to publish step#205

Closed
e7h4n wants to merge 1 commit intomainfrom
fix/npm-publish-token
Closed

ci: pass npm token to publish step#205
e7h4n wants to merge 1 commit intomainfrom
fix/npm-publish-token

Conversation

@e7h4n
Copy link
Copy Markdown
Owner

@e7h4n e7h4n commented Apr 25, 2026

Summary

  • pass the existing NPM_TOKEN secret as NODE_AUTH_TOKEN during the npm publish step
  • fixes the main Changesets workflow failing at publish despite reaching pnpm ci:publish

Checks

  • pnpm lint
  • pnpm test

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@e7h4n
Copy link
Copy Markdown
Owner Author

e7h4n commented Apr 25, 2026

Closing this because the repo is intended to publish via OIDC/trusted publishing; adding NODE_AUTH_TOKEN is not the right fix for that path.

@e7h4n e7h4n closed this Apr 25, 2026
@e7h4n e7h4n deleted the fix/npm-publish-token branch April 25, 2026 14:13
@github-actions
Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 100% 1391 / 1391
🔵 Statements 100% 1391 / 1391
🔵 Functions 100% 122 / 122
🔵 Branches 100% 448 / 448
File CoverageNo changed files found.
Generated in workflow #722 for commit 6d2c6d2 by the Vitest Coverage Report Action

@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 24932794852

Coverage remained the same at 100.0%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1391
Covered Lines: 1391
Line Coverage: 100.0%
Relevant Branches: 378
Covered Branches: 378
Branch Coverage: 100.0%
Branches in Coverage %: Yes
Coverage Strength: 31711.8 hits per line

💛 - Coveralls

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