We'll need to add atom shell and nwjs into our test matrix.

On Sun, Mar 1, 2015, 1:23 PM Steve Smith notifications@github.com wrote:

Hopefully this'll fix the apm install issues.

You can view, comment on, or merge this pull request online at:

#446
Commit Summary

• Upgrade to nan 1.7.0

File Changes

• M package.json
  https://github.com/nodegit/nodegit/pull/446/files#diff-0 (2)

Patch Links:

• https://github.com/nodegit/nodegit/pull/446.patch
• https://github.com/nodegit/nodegit/pull/446.diff

—
Reply to this email directly or view it on GitHub
#446.

Yeah, I still can't get apm install to work pointing to this sha. 😢

apm install worked with that last commit.

👍

On Sun, Mar 1, 2015, 3:12 PM Steve Smith notifications@github.com wrote:

Merged #446 #446.

—
Reply to this email directly or view it on GitHub
#446 (comment).