[BUGFIX] Add back --prefix when installing dependencies #1262
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Apparently, for some reason still unknown to me, when calling
publish
inside a build agent, doing a npm install without the prefix may not create a node_modules folder (could not reproduce in local, but I could on Azure Devops), when you callnpx oc publish
on a folder that does not have a package.json.For this reason I'm adding the option to choose whether to add the
--prefix
or not. It will be added when installing missing dependencies (used by publish and dev), but not when installing the template or the compiler (so it still works correctly without nesting node_modules), because installing the template will always donpm init
first