chore: Use cache in ci.yml

[jongwooo]

Signed-off-by: jongwooo [email protected]

Details

Updated ci.yml to cache dependencies using actions/setup-node. setup-node@v3 or newer has caching built-in.

AS-IS

- name: Install Node.js 18
  uses: actions/setup-node@v3
  with:
    node-version: 18

- name: Install dependencies
  run: npm install

TO-BE

- name: Install Node.js 18
  uses: actions/setup-node@v3
  with:
    node-version: 18
    cache: npm

- name: Install dependencies
  run: npm ci

It’s literally a one line change to pass the cache: npm input parameter.

References

• https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows
• https://thearchivelog.dev/article/caching-dependencies-to-speed-up-workflows/

[simoneb]

@jongwooo as you can see the workflows are failing because npm ci only works with a lockfile, which we don't have in this repo. @RafaelGSS any thoughts? Shall we add it so we can also enable this optimization or are we happy without?

[jongwooo]

@simoneb Oh, I forgot that there was no lockfile...!

[jongwooo]

@jongwooo as you can see the workflows are failing because npm ci only works with a lockfile, which we don't have in this repo. @RafaelGSS any thoughts? Shall we add it so we can also enable this optimization or are we happy without?

If there is no plan to add a lockfile, I will close this PR.

[RafaelGSS]

Usually, we don't use lockfiles on npm modules.

[jongwooo]

@simoneb Thank you for your careful comment! And @RafaelGSS, duly noted. Thank you! I will close this PR.

[RafaelGSS]

Just to expand on that, see: sindresorhus/ama#479 (comment).