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

Specify acceptable types for `lookup` and `dwim` in Reference by rcjsuen · Pull Request #1203 · nodegit/nodegit · GitHub

Repository navigation

Specify acceptable types for lookup and dwim in Reference - #1203

Merged
maxkorp merged 1 commit into
nodegit:masterfrom
rcjsuen:patch-2
Jan 30, 2017
Merged

maxkorp merged 1 commit into
nodegit:masterfrom
rcjsuen:patch-2

Conversation

rcjsuen commented Jan 28, 2017

Copy link
Copy Markdown
Member

According to libgit2's APIs, only git_commits and git_tags are acceptable. Update the documentation to accurately reflect this instead of just taking in an Object.

https://libgit2.github.com/libgit2/#HEAD/group/reset/git_reset

https://libgit2.github.com/libgit2/#HEAD/group/reset/git_reset_default

Make it clear to the client that only Commits and Tags are valid
parameters.

rcjsuen commented Jan 29, 2017

Copy link
Copy Markdown
Member Author

JSHint's character restriction per line meant I had to make more changes to the reset.js beyond just simply modifying the @param tag.

maxkorp commented Jan 30, 2017

Copy link
Copy Markdown
Collaborator

<3

maxkorp merged commit 3e06530 into nodegit:master Jan 30, 2017
rcjsuen deleted the patch-2 branch April 12, 2017 13:01
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