Skip to content
New issue

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

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

Already on GitHub? # to your account

🐛 Use any default branch instead of master #243

Merged
merged 1 commit into from
Mar 31, 2021

Conversation

fregante
Copy link
Contributor

@fregante fregante commented Nov 21, 2020

Replaces and closes #235
Replaces and closes #204
Fixes #207
Fixes #37
Fixes #254
Closes #242

The solution is simple if you detach yourself from master === default and instead use the HEAD as the right terminology in a remote server.

This could alternatively internally referred to as <default> instead of HEAD, but I like minimal-impact PRs.


Also available in the https://github.com/tiged/tiged community fork

@fregante fregante changed the title Use any default branch instead of master 🐛 Use any default branch instead of master Nov 21, 2020
@paulocoghi
Copy link

This also solves one specific scenario explained on #37 (comment)

@rajasegar
Copy link

Any idea when this PR will get merged

@nake89
Copy link

nake89 commented Feb 5, 2021

@fregante Could you modify the readme of your repo to add installation directions. Now it says npm i -g degit. It should say degitto to make it clear. Also, this might just be an issue for me. But after installing I needed to add shebang #!/usr/bin/env node in the beginning of bin.js to be able to run it. We should make your fork a community driven fork, since this project has been abandoned.

@fregante
Copy link
Contributor Author

fregante commented Feb 5, 2021

If anyone would like to maintain a fork, please do. Degitto is meant to be temporary and only fixes this issue. I’ll probably rewrite only the part I need and merge into my GitHub Actions Templating project rather than keep degit/degitto as a dependency.

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

@fregante @paulocoghi @rajasegar @Rich-Harris I started a community driven fork that has this PR included among other fixes. 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).

@Rich-Harris Rich-Harris merged commit bd85ac5 into Rich-Harris:master Mar 31, 2021
@Rich-Harris
Copy link
Owner

Thank you! And apologies for the long delay. Released as 2.8.1

@fregante fregante deleted the patch-1 branch March 31, 2021 17:01
beenotung added a commit to beenotung/degit that referenced this pull request May 12, 2021
@nake89 nake89 mentioned this pull request Mar 6, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
5 participants