FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Make `Reset.fromAnnotated` the same as `Reset.reset` by rcjsuen · Pull Request #1214 · nodegit/nodegit · GitHub

Repository navigation

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

Merged
johnhaley81 merged 1 commit into
nodegit:masterfrom
rcjsuen:optional-reset
Feb 26, 2017
Merged

johnhaley81 merged 1 commit into
nodegit:masterfrom
rcjsuen:optional-reset

Conversation

rcjsuen commented Feb 5, 2017

Copy link
Copy Markdown
Member

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.

Copy link
Copy Markdown
Collaborator

Thanks @rcjsuen!

johnhaley81 merged commit 97ff2cd into nodegit:master Feb 26, 2017
rcjsuen deleted the optional-reset branch February 26, 2017 07:20
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL