Skip to content

Conversation

@pcriadoperez
Copy link
Contributor

@pcriadoperez pcriadoperez commented Oct 22, 2025

Switch to npm following frosty recommendation on commit 085995a

@pcriadoperez pcriadoperez self-assigned this Oct 22, 2025
@pcriadoperez pcriadoperez mentioned this pull request Oct 22, 2025
@carlosmiei carlosmiei requested a review from Copilot October 22, 2025 08:33
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.

Pull Request Overview

This PR migrates the project's package manager from Yarn to npm, updating all relevant scripts and CI configuration files. The change follows a recommendation from a previous commit to standardize on npm.

Key Changes:

  • Replaced yarn commands with npm run equivalents in package.json scripts
  • Updated engine requirements from Yarn to npm
  • Modified CI workflow to use npm commands and package-lock.json for caching

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
package.json Updated npm scripts to use npm run instead of yarn, changed engine requirement from yarn to npm
.github/workflows/ci.yml Replaced yarn commands with npm equivalents and updated cache key to use package-lock.json

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@carlosmiei carlosmiei self-requested a review October 22, 2025 08:34
@carlosmiei carlosmiei merged commit fb571fc into ccxt:master Oct 22, 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