Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update nodegit to version 0.17.0 🚀 #101

Closed
wants to merge 1 commit into from

Conversation

greenkeeperio-bot
Copy link
Contributor

Hello lovely humans,

nodegit just published its new version 0.17.0.

State Update 🚀
Dependency nodegit
New version 0.17.0
Type dependency

This version is not covered by your current version range.

Without accepting this pull request your project will work just like it did before. There might be a bunch of new features, fixes and perf improvements that the maintainers worked on for you though.

I recommend you look into these changes and try to get onto the latest version of nodegit.
Given that you have a decent test suite, a passing build is a strong indicator that you can take advantage of these changes by merging the proposed change into your project. Otherwise this branch is a great starting point for you to work on the update.

Do you have any ideas how I could improve these pull requests? Did I report anything you think isn’t right?
Are you unsure about how things are supposed to work?

There is a collection of frequently asked questions and while I’m just a bot, there is a group of people who are happy to teach me new things. Let them know.

Good luck with your project ✨

You rock!

🌴


GitHub Release

Targeted Platform Changes

In this release we had added support for Node v7 and latest Electron. We have removed support for Node v0.12 and v5.

We are also deprecating nw.js support since it is currently broken, no one in the current team uses it and we would not be able to currently support nw.js in an effective manner with a good user experience.

Now building against shared libcurl lib

For proxy support we now use libcurl for HTTP/HTTPS transport which should have no noticeable change in NodeGit remote operations but if changes are noticed this is worth mentioning as a potential source.

Memory leak fixes and stability increases

Many PR's were made to fix memory leaks as they were found so memory usage should go down in this version for long running scripts. Additionally, when performing operations with callbacks (transfer progress, credentials, etc...) there was a small chance for a segfault when NodeGit would schedule the callback to go into JavaScript. This is now fixed.

Fixes to the build

Many users, especially on windows, were experiencing errors during the build. The build still isn't perfect but a lot of the bugs were fixed.

Bump libgit2 to 0bf0526

The majority of changes to NodeGit v17 were in libgit2. The API breaking changes that are known are:

Summary of changes that were brought in:

#1187 (comment)

Changes to NodeGit outside of libgit2 bump

  • Define GIT_SSH_MEMORY_CREDENTIALS for libgit2 PR #949
  • Add "Path" to ssh variable names in tests for clarity PR #1135
  • Fix revwalk memory leaks PR #1137
  • Fix tree entry leaks PR #1138
  • Fix typo in postinstall script PR #1141
  • Fix windows exception handling in build PR #1143
  • Fix CI failures on node 0.12 PR #1144
  • Fix postinstall script crash when install location has spaces in its path PR #1148
  • Update 0.13 changelog PR #1151
  • Minor documentation fix in Checkout.index PR #1164
  • FreeBSD also uses struct timespec st_mtim PR #1165
  • README.md needs to show where to get Slack invitation PR #1170
  • Add @async tag to Tree#getEntry PR #1178
  • Fix incorrect anchor link in TESTING.md PR #1179
  • Added tests for Tag PR #1180
  • Added tests for Branch PR #1181
  • Escape the spaces in dir for shell command PR #1186
  • Bump libgit to 0bf0526 PR #1187
  • Checkout's tree* functions do not support Oid as a parameter PR #1190
  • Build against shared library for libcurl PR #1195
  • Move libuv calls to correct thread PR #1197
  • Update Repository#createBranch docs PR #1198
  • Remove Node v0.12 and v5 PR #1199
  • Specify acceptable types for lookup and dwim in Reference PR #1203
  • Build for Node 7.4.0 PR #1204
  • Write the index to a repository directly in merge-cleanly.js examples PR #1205

The new version differs by 79 commits .

  • 248afda Add header to separate v0.17 libgit2 changes vs NodeGit specific changes
  • b894ccf Bump to v0.17
  • 5d807eb Update changelog for v0.16.0
  • 4a93dbf Merge pull request #1195 from implausible/feature/enable-curl-support
  • 3828c83 Merge pull request #1197 from srajko/uv_close-fix
  • be4b356 Merge pull request #1187 from srajko/bump-libgit
  • 6d7762b Unreference handle to allow node to terminate
  • 3db424e Bump libgit to 0bf0526
  • 45ce97d Allow both types of failures for invalid credentials test
  • acb6c41 Remove sleep_for_ms
  • 884900f Allocate baton on stack
  • c0c383f Use ExecuteReverseCallback
  • 9c01821 Add support for reverse callbacks to ThreadPool
  • 3e06530 Merge pull request #1203 from rcjsuen/patch-2
  • e359ca5 Merge pull request #1205 from rcjsuen/merge-cleanly

There are 79 commits in total. See the full diff.


✨ Try the all new Greenkeeper GitHub Integration ✨
With Integrations first-class bot support landed on GitHub and we’ve rewritten Greenkeeper to take full advantage of it. Simpler setup, fewer pull-requests, faster than ever.

Screencast

Try it today. Free for private repositories during beta.

@blond blond closed this Mar 12, 2017
@blond blond deleted the greenkeeper-nodegit-0.17.0 branch March 12, 2017 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants