Skip to content

remove node_auth_token from ci#81

Merged
wrongsahil merged 1 commit intomasterfrom
remove-node-auth-token-from-ci
Jan 5, 2026
Merged

remove node_auth_token from ci#81
wrongsahil merged 1 commit intomasterfrom
remove-node-auth-token-from-ci

Conversation

@wrongsahil
Copy link
Copy Markdown
Member

@wrongsahil wrongsahil commented Jan 5, 2026

Summary by CodeRabbit

  • Chores
    • Updated the package publishing workflow to streamline the automated deployment process and enhance security practices. Modified NPM registry configuration and authentication handling in the CI/CD pipeline to ensure reliable and secure package distribution.

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

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 5, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This change modifies the GitHub Actions workflow file for NPM publishing. The setup node.js step now explicitly includes registry-url: 'https://registry.npmjs.org/', while the NODE_AUTH_TOKEN environment variable configuration has been removed from the publish step. This updates how NPM authentication is configured during the CI/CD pipeline.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes


📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5fd3b27 and e3c91fc.

📒 Files selected for processing (1)
  • .github/workflows/publish.yml

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.

@wrongsahil wrongsahil merged commit 68f638b into master Jan 5, 2026
1 of 2 checks passed
@wrongsahil wrongsahil deleted the remove-node-auth-token-from-ci branch January 5, 2026 18:00
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