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
  • !11055

[suggestion] Change "Remove" to "Delete" so as not to be confused with running "yarn remove" to remove the dependency

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Administrator requested to merge github/fork/rpivo/master into main Jun 04, 2021
  • Overview 3
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: rpivo

When going through the steps in the verifyPackageTree.js file, one might try to run yarn remove to remove the dependency, but really the user should manually delete the text entry from the list of [dev] dependencies. If you try yarn remove after deleting yarn.lock file, you will get a "No lockfile in this directory" error (this is what I did -- oops 😅 ).

I think changing "Remove" to "Delete" makes this more clear.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/rpivo/master