Skip to content

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 AI review requested due to automatic review settings April 28, 2025 09:40
Copy link
Contributor

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.

3 participants