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

Specify acceptable types for lookup and dwim in Reference#1203

Merged
maxkorp merged 1 commit intonodegit:masternodegit/nodegit:masterfrom
rcjsuen:patch-2rcjsuen/nodegit:patch-2Copy head branch name to clipboard
Jan 30, 2017
Merged

Specify acceptable types for lookup and dwim in Reference#1203
maxkorp merged 1 commit intonodegit:masternodegit/nodegit:masterfrom
rcjsuen:patch-2rcjsuen/nodegit:patch-2Copy head branch name to clipboard

Conversation

@rcjsuen
Copy link
Member

@rcjsuen rcjsuen commented Jan 28, 2017

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
Copy link
Member Author

rcjsuen commented Jan 29, 2017

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
Copy link
Collaborator

maxkorp commented Jan 30, 2017

<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

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