Skip to content

Use npm install in deploy-docs workflow#1491

Merged
minggangw merged 1 commit intoRobotWebTools:developfrom
minggangw:fix-1489-1
Apr 20, 2026
Merged

Use npm install in deploy-docs workflow#1491
minggangw merged 1 commit intoRobotWebTools:developfrom
minggangw:fix-1489-1

Conversation

@minggangw
Copy link
Copy Markdown
Member

The deploy-docs workflow failed when triggered manually because it relied on npm ci, which requires a committed package-lock.json. This repository intentionally gitignores package-lock.json (see .gitignore), so the install step always errored out with EUSAGE.

Replace npm ci --ignore-scripts with npm install --ignore-scripts --no-audit --no-fund to keep the intent (skip native build scripts, avoid noisy output) while working without a lockfile.

Fix: #1489

Copilot AI review requested due to automatic review settings April 20, 2026 03:06
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the deploy-docs GitHub Actions workflow to install Node dependencies successfully when the workflow is manually triggered, despite the repo intentionally not committing package-lock.json.

Changes:

  • Replace npm ci --ignore-scripts with npm install --ignore-scripts --no-audit --no-fund in the docs deployment workflow.

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

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 85.864%. remained the same — minggangw:fix-1489-1 into RobotWebTools:develop

@minggangw minggangw merged commit ce7e8b4 into RobotWebTools:develop Apr 20, 2026
20 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.

Add GitHub Actions workflow to deploy JSDoc to GitHub

3 participants