wordpress-mobile / release-toolkit-gutenberg-mobile

Automation Scripts for Releasing Gutenberg-Mobile Updates to the WordPress Mobile Apps.
Mozilla Public License 2.0
5 stars 2 forks source link

[CLI] call `nvm use` before each npm call on CI #220

Closed jhnstn closed 8 months ago

jhnstn commented 8 months ago

While testing the prepare command in a Github workflow I noticed that any call to exec.Command seemed to run in a separate process. The result is that nvm needs to be ran before each npm command. Without that the workflow falls back to the default system node.

Testing

The real test is to see if the changes work on a github workflow but we can test locally to make sure it's working.

Steps to check locally: