Skip to content

Remove yarn.lock and update installation instructions to use npm #304

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 28, 2025

Conversation

chrisreddington
Copy link
Contributor

@chrisreddington chrisreddington commented Apr 28, 2025

  1. I removed the yarn.lock file since npm is being used as the package manager in the GitHub Actions workflows. This helps maintain consistency between the local development and CI/CD environments.
  2. I updated the README.md to use npm commands instead of yarn commands. The installation, usage, and build sections now use npm install, npm start, and npm run build respectively, which matches the commands used in the GitHub Actions workflows.

This relates to #272 by standardising on npm for now. And the team can decide whether to transition to other package managers in the future.

…teps used in the GitHub Actions workflow file (using npm install)
@Copilot Copilot AI review requested due to automatic review settings April 28, 2025 09:40
Copy link
Contributor

@Copilot 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

This PR updates the project to use npm as the package manager by removing references to yarn in the README file, aligning local development steps with the CI/CD configurations.

  • Removed yarn commands in the installation, usage, and build sections.
  • Updated documentation to reflect npm commands.

@AndreaGriffiths11 AndreaGriffiths11 merged commit 3cd0b96 into main Apr 28, 2025
4 checks passed
@chrisreddington chrisreddington deleted the update-deps branch April 28, 2025 10:31
@karasowles karasowles mentioned this pull request Apr 28, 2025
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.

2 participants