Skip to content

Conversation

@pandeymangg
Copy link
Contributor

fixes the registry url missing error in the release action

@pandeymangg pandeymangg requested a review from Dhruwang October 1, 2025 07:19
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 1, 2025

@pandeymangg pandeymangg added this pull request to the merge queue Oct 1, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 1, 2025

Walkthrough

The release workflow at .github/workflows/release.yml was updated to set an explicit npm registry for the Node.js setup step. Specifically, the Setup Node.js 22.x action now includes registry-url: "https://registry.npmjs.org". No other steps, control flow, or error handling were modified. No exported or public declarations were affected.

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title directly references the primary change—fixing the release action’s registry URL—and concisely summarizes the main update without irrelevant details.
Description Check ✅ Passed The description clearly explains that the pull request addresses the missing registry URL error in the release action, directly relating to the changeset.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 676982b and 19ca155.

📒 Files selected for processing (1)
  • .github/workflows/release.yml (1 hunks)
🔇 Additional comments (1)
.github/workflows/release.yml (1)

30-30: Explicit registry fixes npm publish

Setting registry-url on the setup-node step ensures the later npm publish resolves the registry without relying on ambient config. This should eliminate the missing-registry failure.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Merged via the queue into main with commit ffec69f Oct 1, 2025
9 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.

3 participants