Avoid using IIFE to require scrypt

[ppoliani]

This should fix #3018

[coveralls]

Coverage remained the same at 83.191% when pulling ca845c6 on ppoliani:ch-use-static-import into 2d0a3f3 on ethereum:1.x.

[ppoliani]

Not sure why the build fails. All tests seem to pass. Is there something that is missing?

[michaelsbradleyjr]

@ppoliani the current build pipeline doesn't deal well with static require in reference to a package that may be supplied by the person who installed it alongside web3, i.e. the scrypt package is no longer a dependency of web3 itself. That's why I introduced the dynamic require. The webpack warning that introduces doesn't cause builds to fail strictly speaking; however some developers' build pipelines may be setup to fail outright even on a warning.

See: #2938 (comment).

A better solution is in the works, hopefully I'll get the PR up tomorrow.

[nivida]

Closed because of the newly opened PR‘s: #3036 #3035