Skip to content

ci: use npm ci for deterministic installs - #133

Merged
cb-sriramthiagarajan merged 1 commit into
masterfrom
chore/ci-install
Aug 14, 2026
Merged

ci: use npm ci for deterministic installs#133
cb-sriramthiagarajan merged 1 commit into
masterfrom
chore/ci-install

Conversation

@cb-karthikp

@cb-karthikp cb-karthikp commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Use npm ci for deterministic dependency installation in the Makefile install target and the prepack script.

@snyk-io

snyk-io Bot commented Aug 14, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues
Secrets 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: e9f1aa3d-a4e4-4ec5-b855-d4bb4ab8d00f

📥 Commits

Reviewing files that changed from the base of the PR and between b74a74e and 39a16b5.

📒 Files selected for processing (2)
  • Makefile
  • package.json

Walkthrough

The installation and packaging workflows now use npm ci instead of npm install.

Changes

Deterministic npm installation

Layer / File(s) Summary
Update installation workflows
Makefile, package.json
The install target and prepack script now run npm ci.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 39a16

This PR makes CI installs more deterministic without any identified merge-blocking risk; it is merge-ready after normal checks and review.

Suggested reviewers: cb-alish

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@cb-karthikp
cb-karthikp marked this pull request as ready for review August 14, 2026 14:27
@cb-sriramthiagarajan
cb-sriramthiagarajan merged commit 853e63f into master Aug 14, 2026
9 checks passed
@cb-sriramthiagarajan
cb-sriramthiagarajan deleted the chore/ci-install branch August 14, 2026 14:55
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