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

Upgrade to newest version of level dependencies. #5

Closed
wants to merge 3 commits into from

Conversation

typpo
Copy link
Contributor

@typpo typpo commented Jun 2, 2017

Fixes #4 - npm install fails on some newer versions of node. Tested by npm installing successfully, and then npm run test.

I also bumped the minor version, but let me know if you'd prefer to do that yourself when you publish it.

typpo added 2 commits June 1, 2017 22:22
Fixes issue where level-dump package would not install successfully, thlorenz#4
@thlorenz
Copy link
Owner

thlorenz commented Jun 2, 2017

Please remove the last commit, I have a script that does the versioning + tagging + publishes to npm.

@typpo
Copy link
Contributor Author

typpo commented Jun 2, 2017

Done

@thlorenz
Copy link
Owner

thlorenz commented Jun 2, 2017

OK you added another commit instead of removing the other one .. I adapted to what I actually wanted and published as 1.1.
Thanks for your help.

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.

None yet

2 participants