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

Make Reset.fromAnnotated the same as Reset.reset#1214

Merged
johnhaley81 merged 1 commit intonodegit:masternodegit/nodegit:masterfrom
rcjsuen:optional-resetrcjsuen/nodegit:optional-resetCopy head branch name to clipboard
Feb 26, 2017
Merged

Make Reset.fromAnnotated the same as Reset.reset#1214
johnhaley81 merged 1 commit intonodegit:masternodegit/nodegit:masterfrom
rcjsuen:optional-resetrcjsuen/nodegit:optional-resetCopy head branch name to clipboard

Conversation

@rcjsuen
Copy link
Member

@rcjsuen rcjsuen commented Feb 5, 2017

The fromAnnotated function should be asynchronous in addition to having its checkout_opts parameter be optional as the function delegates to the same function as reset internally in libgit2.

The reset function is asynchronous and also flags its checkout
options as being optional. fromAnnotated should be updated
accordingly as the two functions basically do the same thing.
@johnhaley81
Copy link
Collaborator

Thanks @rcjsuen!

@johnhaley81 johnhaley81 merged commit 97ff2cd into nodegit:master Feb 26, 2017
@rcjsuen rcjsuen deleted the optional-reset branch February 26, 2017 07:20
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.

2 participants

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