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

Remove `Index#addAll` hack by johnhaley81 · Pull Request #875 · nodegit/nodegit · GitHub

Repository navigation

Remove Index#addAll hack - #875

Merged
johnhaley81 merged 2 commits into
masterfrom
remove-index-add-all-hack
Jan 21, 2016
Merged

johnhaley81 merged 2 commits into
masterfrom
remove-index-add-all-hack

Conversation

Copy link
Copy Markdown
Collaborator

Now that libgit2/libgit2#2687 has landed we don't need our hack to speed up Index#addAll anymore.

This fixes #751

John Haley added 2 commits January 20, 2016 23:43
Now that libgit2/libgit2#2687 has landed we don't need our hack to speed up `Index#addAll` anymore.

This fixes #751
johnhaley81 added a commit that referenced this pull request Jan 21, 2016
johnhaley81 merged commit af384b4 into master Jan 21, 2016
johnhaley81 deleted the remove-index-add-all-hack branch January 21, 2016 21:55
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.

libgit2 bug with workaround in index.addAll() is now fixed, we should update index.js

1 participant


Back | FazBrowse Home | New Git URL