Skip to content

Stick to npm as the single app package manager#78

Open
jddark62 wants to merge 1 commit into
manikyabard:masterfrom
jddark62:bounty-20-single-package-manager
Open

Stick to npm as the single app package manager#78
jddark62 wants to merge 1 commit into
manikyabard:masterfrom
jddark62:bounty-20-single-package-manager

Conversation

@jddark62

@jddark62 jddark62 commented Jun 5, 2026

Copy link
Copy Markdown

Closes #20.

Summary

  • Removed app/yarn.lock so the app no longer carries both npm and Yarn lockfiles.
  • Kept the existing npm workflow intact because install.sh and the package scripts already use npm install, npm start, and npm run build.
  • Added a short README note telling contributors to keep app/package-lock.json as the single lockfile.

Verification

  • git diff --check
  • Confirmed app/yarn.lock is removed.
  • Parsed app/package.json and app/package-lock.json successfully with Node.js.
  • Searched README, package scripts, and install script for yarn.lock, package-lock, and npm references.

I could not run npm install or the app build in this workspace because npm/yarn/pnpm/npx are not installed here. The change is limited to removing the unused competing lockfile and documenting the package-manager choice.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This pull request standardizes the frontend app’s package-management approach by committing to npm as the single package manager and documenting that choice for contributors, addressing Issue #20.

Changes:

  • Removed the competing Yarn lockfile (app/yarn.lock) so the app isn’t tracked by two lockfiles.
  • Documented that app/package-lock.json should remain the single lockfile and contributors should not regenerate app/yarn.lock.

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

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.

Stick to one Package Manager

2 participants