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 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants