Add the web config dependent module installation command to the Makefile

[ren510dev]

What would you like to be added:

I propose to add yarn install to the following Makefile.

• https://github.com/pipe-cd/pipecd/blob/master/Makefile#L110-L111

Why is this needed:

According to this document, to install C-Plane in the local environment, make run/pipecd must be run.
However, the one-liner fails to run because there are no dependent modules in the ./web directory.

[webpack-cli] Failed to load '/Users/ren/tmp/pipecd/web/webpack.config.js' config
[webpack-cli] Error: Cannot find module 'dotenv'
Require stack:
- /Users/ren/tmp/pipecd/web/webpack.config.js
- /Users/ren/.nvm/versions/node/v20.5.0/lib/node_modules/webpack-cli/lib/webpack-cli.js
- /Users/ren/.nvm/versions/node/v20.5.0/lib/node_modules/webpack-cli/lib/bootstrap.js
- /Users/ren/.nvm/versions/node/v20.5.0/lib/node_modules/webpack-cli/bin/cli.js
- /Users/ren/.nvm/versions/node/v20.5.0/lib/node_modules/webpack/bin/webpack.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1048:15)
    at Module._load (node:internal/modules/cjs/loader:901:27)
    at Module.require (node:internal/modules/cjs/loader:1115:19)
    at require (node:internal/modules/helpers:119:18)
    at Object.<anonymous> (/Users/ren/tmp/pipecd/web/webpack.config.js:3:1)
    at Module._compile (node:internal/modules/cjs/loader:1233:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1287:10)
    at Module.load (node:internal/modules/cjs/loader:1091:32)
    at Module._load (node:internal/modules/cjs/loader:938:12)
    at Module.require (node:internal/modules/cjs/loader:1115:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/Users/ren/tmp/pipecd/web/webpack.config.js',
    '/Users/ren/.nvm/versions/node/v20.5.0/lib/node_modules/webpack-cli/lib/webpack-cli.js',
    '/Users/ren/.nvm/versions/node/v20.5.0/lib/node_modules/webpack-cli/lib/bootstrap.js',
    '/Users/ren/.nvm/versions/node/v20.5.0/lib/node_modules/webpack-cli/bin/cli.js',
    '/Users/ren/.nvm/versions/node/v20.5.0/lib/node_modules/webpack/bin/webpack.js'
  ]
}
error Command failed with exit code 2.

Therefore, I propose to add yarn install to the Makefile. Wdyt ?

[hungran]

+1 on this,
I was tried to build image arm64 and stuck when make build/web doesn't have a step yarn install hence did it by manually

[khanhtc1202]

@GotoRen @hungran we have make update/web-deps which is preferred to be run once before run/build web with commands. Of course I understand that in some cases, devs only want to run single command for all but just want to share that we let it be that way currently because we handle that in different way. If both feel it's better to make that web deps installation part of other command, then feel free to go (with voting). But I'm still think that we should keep dependencies installation separated from building commands, due to we can cache dependencies installation layer, but always want to building layer be rerun each time, wdyt?

[khanhtc1202]

If it was a docs miss, let's update docs to mention make update/web-deps command.

[hungran]

Maksense!!!
I think just update the document

[ren510dev]

Me too!! I agree to update the document.
I will work on improving this issue.

[khanhtc1202]

Thanks @GotoRen @hungran