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

updated npm depricated version to updated one #886

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abdulhaqshah
Copy link
Contributor

didn't updated the hapi because this need code changes and the latest version have an error that's why.

@gr2m
Copy link
Member

gr2m commented Oct 21, 2019

As we update the dependencies, we should also drop support for outdated Node versions, which would be a breaking version. I don't mind doing that, upgrading should be easy enough.

Can you update

hoodie/package.json

Lines 52 to 54 in 159f339

"engines": {
"node": ">=6"
},

(Bump it to 8 for now)

And the .travis.yml file to test in Node 12 (which is LTS now)?

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