Skip to content

Change npm install to npm ci#111

Merged
aleks-pro merged 5 commits into
masterfrom
aleks-pro-patch-1
Jun 24, 2026
Merged

Change npm install to npm ci#111
aleks-pro merged 5 commits into
masterfrom
aleks-pro-patch-1

Conversation

@aleks-pro

Copy link
Copy Markdown
Contributor

No description provided.

Updated Docker commands to use 'npm ci' instead of 'npm i' for consistent dependency installation.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates installation steps to use npm ci for more deterministic dependency installs in CI and in the Kerberos auth detached example’s Docker containers.

Changes:

  • Switch GitHub Actions workflow dependency installation from npm install to npm ci.
  • Update detached-examples/kerberos-auth launcher to copy package-lock.json into server/client containers and run npm ci instead of npm i.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
detached-examples/kerberos-auth/index.js Copies lockfiles into Docker containers and uses npm ci for deterministic installs.
.github/workflows/main.yml Uses npm ci in CI workflow instead of npm install.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/main.yml
Comment thread detached-examples/kerberos-auth/index.js
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@aleks-pro aleks-pro changed the title Change npm install to npm ci in workflow Change npm install to npm ci Jun 24, 2026
@aleks-pro aleks-pro merged commit 2f1ac3d into master Jun 24, 2026
4 of 5 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.

4 participants