Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix new project not created if a non-empty folder with the same name as boilerplate repo exists. #88

Merged
merged 1 commit into from
Feb 28, 2019

Conversation

anto-christo
Copy link
Contributor

Hi,
I came across this issue today. While creating a new project, if a non-empty folder with the same name as the base boilerplate repository name (mevn-boilerplate/mevn-pwa-boilerplate..) exists in the same location, then the new project is not created and the folder with boilerplate name is renamed to the new project name.

This is happening because we are first cloning the boilerplate repo and then renaming it. So, while cloning, it detects that a non-empty folder with the same name already exists and it stops cloning. The renaming is then done on the existing folder.

Fix

This can be fixed by cloning the boilerplate repo directly with the new project name instead of the repo name. This way we eliminate the need for renaming as well.

This can be done by passing the project name as an additional option to git clone command.
Eg: git clone url project_name

…as the boilerplate already exists in same directory
@welcome
Copy link

welcome bot commented Feb 28, 2019

Thanks for opening this pull request! Please check out our contributing guidelines.

@jamesgeorge007 jamesgeorge007 added the bug Something isn't working label Feb 28, 2019
@jamesgeorge007 jamesgeorge007 added this to In progress in Roadmap via automation Feb 28, 2019
@jamesgeorge007 jamesgeorge007 modified the milestones: v1.0.8, v1.0.9 Feb 28, 2019
Copy link
Member

@jamesgeorge007 jamesgeorge007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@jamesgeorge007 jamesgeorge007 merged commit dc8ee4b into madlabsinc:master Feb 28, 2019
Roadmap automation moved this from In progress to Done Feb 28, 2019
@welcome
Copy link

welcome bot commented Feb 28, 2019

Hurray 🎉 Congrats on your very first contribution 👏

@jamesgeorge007 jamesgeorge007 removed this from Done in Roadmap Feb 28, 2019
@jamesgeorge007 jamesgeorge007 added this to In progress in Roadmap via automation Mar 1, 2019
@jamesgeorge007 jamesgeorge007 moved this from In progress to Done in Roadmap Mar 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Roadmap
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants