Follow-up: Remove sed command that overwrites compiled VSCode Workbench module
MR: refactor: handle packaged node_modules (!400 - merged)
The following discussion from !394 (merged) should be addressed:
-
@pslaughter started a discussion: (+4 comments) question: Oh man, this one is tricky :/
I'm really nervous about string replacing something across the compiled
.js. It's a big assumption that this is safe... What if we didn't try to move things over tobundled_node_modules? It's probably safer if we try to preserve the structure of the package. WDYT? Maybe we can try pairing on this?
Edited by Enrique Alcántara