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 for node 6 and lodash v4 #800

Closed
wants to merge 5 commits into from
Closed

Update for node 6 and lodash v4 #800

wants to merge 5 commits into from

Conversation

intelliot
Copy link
Collaborator

@intelliot intelliot commented Sep 25, 2017

Also enables ripple-lib to work with node 6

Updating lodash resolves the following error

> babel -D --optional runtime -d dist/npm/ src/

module.js:471
    throw err;
    ^

Error: Cannot find module 'lodash/kebabCase'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
...

@intelliot intelliot changed the title Update lodash to 4 Update to use lodash v4 Sep 25, 2017
Copy link
Collaborator

@wilsonianb wilsonianb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider folding that last commit into the first.

@@ -1,11 +1,10 @@
machine:
node:
version: 0.12.0
version: 6.11.3
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should these docs which currently say 6.9.0 be updated?
#760

- circle.yml - use node 6
- Fix eslint errors
- Update coveralls and add path to babel
- Update ripple-lib-transactionparser
@intelliot
Copy link
Collaborator Author

@wilsonianb Comments addressed - thanks!

@intelliot intelliot changed the title Update to use lodash v4 Update for node 6 and lodash v4 Sep 27, 2017
Copy link
Collaborator

@wilsonianb wilsonianb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@intelliot
Copy link
Collaborator Author

Rebased and merged with f42dafd

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