Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Conversation

@lbertenasco
Copy link
Contributor

@lbertenasco lbertenasco commented Jun 18, 2019

Credits to #2386

@coveralls
Copy link

coveralls commented Jun 18, 2019

Coverage Status

Coverage decreased (-0.06%) to 95.682% when pulling 0a69f6c on lbertenasco:fix/optional-scrypt.js into 8ed2fbc on ethereum:2.x.

@SebastianGiro SebastianGiro mentioned this pull request Jun 19, 2019
@SebastianGiro
Copy link

@nivida This fixes the scrypt problem with node 12 for me

@SebastianGiro SebastianGiro mentioned this pull request Jul 1, 2019
@nivida nivida added 2.x 2.0 related issues Bug Addressing a bug In Progress Currently being worked on labels Jul 8, 2019
@nivida
Copy link
Contributor

nivida commented Jul 12, 2019

I'm closing this PR because of the newly proposed and improved solution from these PRs:
#2938
#2937

@nivida nivida closed this Jul 12, 2019
@nivida nivida removed the In Progress Currently being worked on label Jul 12, 2019
@lbertenasco lbertenasco deleted the fix/optional-scrypt.js branch July 12, 2019 14:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

2.x 2.0 related issues Bug Addressing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants