Skip to content

Fix npm publish#13

Merged
nastassiafulconis merged 7 commits into
mainfrom
fix-npm-publish
Nov 6, 2025
Merged

Fix npm publish#13
nastassiafulconis merged 7 commits into
mainfrom
fix-npm-publish

Conversation

@nastassiafulconis
Copy link
Copy Markdown
Collaborator

No description provided.

- Remove registry-url from setup-node (causes conflicts)
- Add explicit .npmrc configuration step with NPM_TOKEN
- Matches pattern from working type-graphql release workflow
- Add npm update step to ensure npm 11.5.1+ is installed
- Fix token variable escaping in .npmrc (remove backslash)
- This should properly write the actual token value instead of literal ${NPM_TOKEN}
@nastassiafulconis nastassiafulconis requested review from a team and benminer November 6, 2025 18:19
@nastassiafulconis nastassiafulconis enabled auto-merge (squash) November 6, 2025 18:19
@nastassiafulconis nastassiafulconis merged commit 3432ff6 into main Nov 6, 2025
6 checks passed
@nastassiafulconis nastassiafulconis deleted the fix-npm-publish branch November 6, 2025 18:35
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