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

2797 : Improve untracked files message during eject

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/Dubes/2797-improve-eject-dirty-git-error-message into master Jul 22, 2017
  • Overview 8
  • Commits 2
  • Pipelines 0
  • Changes 1

Created by: Dubes

This attempts to improve the message shown to user when ejecting on a git repo having untracked changes ( Issue: https://github.com/facebookincubator/create-react-app/issues/2797 )

Proposed message

Your git working tree is dirty. This will make it hard for you to resolve any unintended consequences eject may have on your changes. Please use git status to verify the changed files, then either remove , stash or commit them before trying again.

Message break down:

  • state the root cause
  • state why this is not allowed
  • hint for the user to identify files causing problems
  • hint for the user towards action they can commit

I did not list the files (as was done previously) as that may lead users to more confusion (why is CRA having problem with this file etc.), instead tell the users how they can find it.

Testing: Screenshot attached: image

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/Dubes/2797-improve-eject-dirty-git-error-message