Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • C create-react-app
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,547
    • Issues 1,547
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 417
    • Merge requests 417
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Meta
  • create-react-app
  • Merge requests
  • !11322

Use env var to detect yarn or npm as the package manager

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/lukekarrys/lk/use-npm into main Aug 18, 2021
  • Overview 13
  • Commits 5
  • Pipelines 0
  • Changes 7

Created by: lukekarrys

This removes the --use-npm flag and adds the --use-yarn flag. It also removes the previous behavior of checking for the existence of yarn as the signal to use it as the package manager. The new behavior will be to use npm by default and to use yarn with the --use-yarn flag.

The goal of this PR is to remove the check on whether yarn is present in the user's path as the signal to use yarn.

This PR is implemented on top of #11304, since it changed some of the same files and was easier to get tests passing on top of that. I will rework this based on the outcome of that PR. For now, here's the differing commits in this PR: https://github.com/lukekarrys/create-react-app/compare/lk/dev-npm-7...lk/use-npm

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/lukekarrys/lk/use-npm