Skip to content

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

Specify acceptable types for lookup and dwim in Reference#1203
maxkorp merged 1 commit into
nodegit:masterfrom
rcjsuen:patch-2

Conversation

@rcjsuen

@rcjsuen 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

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

maxkorp commented Jan 30, 2017

Copy link
Copy Markdown
Collaborator

<3

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