Use Browserify 17
Opened this issue · 1 comments
Ideally, Mochify would use Browserify 17 (instead of 16 at the time of writing) to keep in sync with maintained versions of the underlying tools.
Picking up the comment here: #223 (comment) I looked into why such an update from 16 to 17 is breaking Mochify and found out the following (this issue is mostly about collecting what I found out for whenever this updates becomes required for some reason):
- There is yet another new deprecation warning about the usage of
SharedArrayBuffer
that littersstderr
, but that can be swallowed just as Mochify already does with other deprecation messages. (browserify/node-util#58) - After muting these warnings, the tests for
--no-detect-globals
fail because of the following:
Browserify 17 updatesutil
to 0.12 browserify/browserify#1844, containing this commit browserify/node-util@6b82825 which is readingprocess.env.NODE_DEBUG
without checking whetherprocess
is even defined. This means that when passing the option not to shim globals to browserify, any code that requiresutil
(Mocha for example) will fail with anUncaught ReferenceError: process is not defined
I'm not entirely sure if this is something that could be or should be worked around in Mochify or if this is something that is to be considered an upstream bug in either node-util
or browserify
. Mochify could of course also drop the --no-detect-globals
flag again but this would mean #208 comes back to life.
There's an issue in node-util
about this browserify/node-util#43