Title: Commit.getParents working with merge commits by bjornarg · Pull Request #357 · nodegit/nodegit · GitHub
Open Graph Title: Commit.getParents working with merge commits by bjornarg · Pull Request #357 · nodegit/nodegit
X Title: Commit.getParents working with merge commits by bjornarg · Pull Request #357 · nodegit/nodegit
Description: getParents failed when more than one commit was present, since it attempted to get the second parent of commit's parent instead of it's own second parent. The promise for the second parent was also pushed onto the array for Promise.all() too late to actually have an effect. Limit was not working properly as a limit, since it attempted to retrieve the set limit regardless of how many parents a commit had, failing if the limit was set to more than Commit.parentcount().
Open Graph Description: getParents failed when more than one commit was present, since it attempted to get the second parent of commit's parent instead of it's own second parent. The promise for the second parent ...
X Description: getParents failed when more than one commit was present, since it attempted to get the second parent of commit's parent instead of it's own second parent. The promise for the second...
Opengraph URL: https://github.com/nodegit/nodegit/pull/357
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:d1d85ee3-98fd-f820-d959-55fdd5c64e13 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 9FAE:338491:196223:2135F8:6A5FE912 |
| html-safe-nonce | 26ca0f5b58b4d8f44ab91475271d8c2cfde36844b84a8c8eda5361b5f9b2bce0 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5RkFFOjMzODQ5MToxOTYyMjM6MjEzNUY4OjZBNUZFOTEyIiwidmlzaXRvcl9pZCI6IjI3NzQwNzkyNjc1NzE0OTMxMzgiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 286b0af3ebf47ad445bedebbfcf63e1e0bf968d80e11eae4b4fab2f280f145a3 |
| hovercard-subject-tag | pull_request:27578683 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/nodegit/nodegit/pull/357/files |
| twitter:image | https://avatars.githubusercontent.com/u/275651?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/275651?s=400&v=4 |
| og:image:alt | getParents failed when more than one commit was present, since it attempted to get the second parent of commit's parent instead of it's own second parent. The promise for the second parent ... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | ba296ecd3a3521238fe43b3b3b08cc09207ddc21173e16ee656db03803de8e2b |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/nodegit/nodegit git https://github.com/nodegit/nodegit.git |
| octolytics-dimension-user_id | 657068 |
| octolytics-dimension-user_login | nodegit |
| octolytics-dimension-repository_id | 1383170 |
| octolytics-dimension-repository_nwo | nodegit/nodegit |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1383170 |
| octolytics-dimension-repository_network_root_nwo | nodegit/nodegit |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 6e50fb37e29e4aa0df6386cb485e58b0e3ca97ab |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width