Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Better cloning with NodeGit#453

Merged
tbranyen merged 4 commits intomasternodegit/nodegit:masterfrom
better-clonenodegit/nodegit:better-cloneCopy head branch name to clipboard
Mar 2, 2015
Merged

Better cloning with NodeGit#453
tbranyen merged 4 commits intomasternodegit/nodegit:masterfrom
better-clonenodegit/nodegit:better-cloneCopy head branch name to clipboard

Conversation

@tbranyen
Copy link
Member

@tbranyen tbranyen commented Mar 2, 2015

Right now we have the current call mechanism for clone:

var Git = require('nodegit');

Git.Clone.clone('https://github.com/nodegit/nodegit', 'nodegit').then(function(repo) {
  /* work with the repository */
});

This is pretty verbose, compared to:

var Git = require('nodegit');

Git.Clone('https://github.com/nodegit/nodegit', 'nodegit').then(function(repo) {
  /* work with the repository */
});

So I've fixed that in the JS-side. The top-level NodeGit.Clone Object is now a Function that behaves like an Object.

This is not a backwards-breaking change, so hopefully we're all in favor?

tbranyen
tbranyen reviewed Mar 2, 2015
View reviewed changes
test/tests/clone.js Outdated
Copy link
Member Author

@tbranyen tbranyen Mar 2, 2015

Choose a reason for hiding this comment

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

Damnit!

tbranyen added 4 commits March 2, 2015 17:48
@tbranyen
Convert Clone from object to function
efb2376 
I find it really annoying that we have to do `NodeGit.Clone.clone` just
to clone a repository.  It would be much nicer to get a similar effect
like in C, where you can run `git_clone`.  This copies over the effect
of an object, but actually using our `clone` patched function instead.

I've made this commit first to show it does not introduce regressions
with our current clone code.
@tbranyen
Test showing shorter clone syntax
5773320
@tbranyen
Updated examples to show new syntax
7061d68
@tbranyen
Remove only from describe block
4c888f5
@johnhaley81
Copy link
Collaborator

johnhaley81 commented Mar 2, 2015

This flows much better than NodeGit.Clone.clone

👍

@tbranyen
Copy link
Member Author

tbranyen commented Mar 2, 2015

Cool, merging!

tbranyen added a commit that referenced this pull request Mar 2, 2015
@tbranyen
Merge pull request #453 from nodegit/better-clone
22430ec 
Better cloning with NodeGit
@tbranyen tbranyen merged commit 22430ec into master Mar 2, 2015
@tbranyen tbranyen deleted the better-clone branch March 2, 2015 23:00
@johnhaley81 johnhaley81 added this to the 0.3.0 milestone Mar 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

0.3.0

Development

Successfully merging this pull request may close these issues.

2 participants

@tbranyen @johnhaley81
Morty Proxy This is a proxified and sanitized view of the page, visit original site.