Skip to content

Add package-lock.json for CI npm caching#19

Merged
hotlong merged 2 commits intomainfrom
copilot/update-github-actions-configuration
Feb 2, 2026
Merged

Add package-lock.json for CI npm caching#19
hotlong merged 2 commits intomainfrom
copilot/update-github-actions-configuration

Conversation

Copy link
Contributor

Copilot AI commented Feb 2, 2026

CI workflow fails at setup-node step because cache: 'npm' requires a lock file, but package-lock.json was gitignored.

Changes

  • Remove package-lock.json from .gitignore
  • Add generated package-lock.json with resolved dependencies

The workflow uses npm ci which requires a lock file for reproducible installs. Standard practice for Node.js projects is to commit lock files.

Original prompt

引用: https://github.com/objectstack-ai/objectstack-starter/actions/runs/21578484575/job/62170796149#step:3:1


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Remove package-lock.json from .gitignore to enable npm caching in CI workflow

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copilot AI changed the title [WIP] Update GitHub Actions configuration Add package-lock.json for CI npm caching Feb 2, 2026
Copilot AI requested a review from hotlong February 2, 2026 06:25
@hotlong hotlong marked this pull request as ready for review February 2, 2026 06:26
Copilot AI review requested due to automatic review settings February 2, 2026 06:26
Copy link

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.

Copilot wasn't able to review any files in this pull request.


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

@hotlong hotlong merged commit 421c1a5 into main Feb 2, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants