Skip to content

Regenerated package-lock#12

Merged
mbwatson merged 5 commits intostagecc:mainfrom
Amaru333:main
Mar 13, 2026
Merged

Regenerated package-lock#12
mbwatson merged 5 commits intostagecc:mainfrom
Amaru333:main

Conversation

@Amaru333
Copy link
Copy Markdown
Collaborator

Regenerated the package-lock.json file to ensure that there's no errors while running it locally on Mac systems.

Copy link
Copy Markdown
Member

@mbwatson mbwatson left a comment

Choose a reason for hiding this comment

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

looks good. builds fine for me.

Copy link
Copy Markdown
Collaborator

@suejinkim20 suejinkim20 left a comment

Choose a reason for hiding this comment

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

👍🏼 works for me as well!

@mbwatson mbwatson self-requested a review March 12, 2026 14:23
Copy link
Copy Markdown
Member

@mbwatson mbwatson left a comment

Choose a reason for hiding this comment

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

I think we'd benefit from a short note in the README, something like:

Always use npm ci to install dependencies locally. Only use npm install <package> when adding/removing a dependency, and commit the updated package-lock.json. Running bare npm install can cause platform-specific drift in the lock file.

maybe documenting this convention prevents future headaches. what do you think?

Copy link
Copy Markdown
Member

@mbwatson mbwatson left a comment

Choose a reason for hiding this comment

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

looks good! the new build check seems good, and the documentation will be helpful moving forward. 👍

@mbwatson mbwatson merged commit 8c8482f into stagecc:main Mar 13, 2026
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.

3 participants