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

Document usage for gitlab/bitbucket #16

Closed
wants to merge 1 commit into from
Closed

Document usage for gitlab/bitbucket #16

wants to merge 1 commit into from

Conversation

IvanSanchez
Copy link

With support for repos other than github merged in #13, it should be nice to add a few lines to the readme to explain that functionality

@Rich-Harris
Copy link
Owner

ah, whoops, sorry — didn't realise you'd sent this PR, I just updated the README. It actually doesn't support custom git servers, because I'm not sure if there's a standard way to generate the .tar.gz URL. Would be glad to be enlightened though!

@Rich-Harris Rich-Harris closed this Aug 5, 2017
@melniker
Copy link
Contributor

melniker commented Aug 6, 2017

@Rich-Harris, I was planning on working on non-github support before I saw the issues closed today, and my plan included a config json file that gave the characteristics of the servers. I was going to pre-populate it for github/gitlab/butbucket but it could then be easily extendable for other servers. It would specify every property you'd need, including URI format & location of the tarball.

This way, degit would be easily extendable not only for public git servers (which could be put into debit/master via PR) but also for private instances inside a big corporation, e.g.

Could also of course take these config settings via cli parameters.

Thoughts?

@Rich-Harris
Copy link
Owner

@melniker I like it! That sounds great

@IvanSanchez IvanSanchez deleted the patch-1 branch August 7, 2017 07:48
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