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

Check for presence of folders before continuing eject.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/heldinz/master into master Oct 24, 2016
  • Overview 4
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: heldinz

Verifies that the folders created during an eject (not just the files) do not exist. Exits with error if they do exist.

I tested this by creating scripts and config/jest folders in the app, then running npm run eject. The script failed early on both scripts and config, as expected. Removing these directories allowed the script to continue running and successfully create the required directories and copy the files over to them.

Closes #939 (closed).

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