Closed timfish closed 1 week ago
We did get some feedback about this warning. Do we perhaps want to add the functionality hide the warnings behind an environmental variables?
Maybe safer to have DISABLE_IITM_WARNINGS
or something similar.
Do not use NODE_ENV
. I provided feedback in #112 that we should use Node.js's native feature for issuing warnings.
Since #106 might not get merged, I cherry picked the improvements to logged error messages when module wrapping fails due to parsing failures.
processModule
may parse multiple files in the process of wrapping a module so it's important to log the exact file the fails parsing.