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

Use rimrafSync instead of rm -rf in --mode=git #191

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kling90
Copy link

@kling90 kling90 commented Apr 15, 2020

I get the following error when using --mode=git with a private repository in Windows (PowerShell):

! Command failed: rm -rf E:\dev\test\.git
'rm' is not recognized as an internal or external command,
operable program or batch file.

Since rm is not avaliable in PowerShell and I saw that you were already using rimraf you could use that instead and not be tied to a unix environment.

@rowanparker
Copy link

@Rich-Harris any chance of accepting this?

nake89 added a commit to tiged/tiged that referenced this pull request Feb 11, 2021
nake89 added a commit to tiged/tiged that referenced this pull request Feb 11, 2021
* main and other default branches work, not just master Rich-Harris/degit#243
* Use rimrafSync instead of rm -rf in --mode=git Rich-Harris/degit#191
* Updated dependencies
* Forked to tiged, community driven fork of degit
@nake89
Copy link

nake89 commented Feb 11, 2021

@kling90 @rowanparker
I started a community driven fork that has this PR included among other fixes/features (works automatically with main branch). https://github.com/tiged/tiged
I welcome people to star it, if they like it (currently has 1).
When or if Rich Harris becomes active again, I am willing to give him control of that repo or merge it to this.
In the mean time I am looking for people to maintain this repo. Any help is welcome.

If you want to use the fork. Simply run:

npm uninstall -g degit
npm install -g tiged

Then you can just use it like you used to. degit username/repo my-app. You don't need to write tiged (it doesn't even work with that name).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants