Skip to content

chore: Use cache in check-dist.yml#93

Merged
yoannchaudet merged 1 commit intoactions:mainfrom
jongwooo:chore/use-cache-in-check-dist
Jan 31, 2023
Merged

chore: Use cache in check-dist.yml#93
yoannchaudet merged 1 commit intoactions:mainfrom
jongwooo:chore/use-cache-in-check-dist

Conversation

@jongwooo
Copy link
Contributor

Signed-off-by: jongwooo [email protected]

Description

Updated check-dist.yml to cache dependencies using actions/setup-node.

AS-IS

- name: Set Node.js 16.x
  uses: actions/setup-node@v3
  with:
    node-version: 16.x

TO-BE

- name: Set Node.js 16.x
  uses: actions/setup-node@v3
  with:
    node-version: 16.x
    cache: npm

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

@jongwooo jongwooo requested a review from a team as a code owner January 23, 2023 15:58
@yoannchaudet
Copy link
Contributor

Thank you, it looks great to me!

@yoannchaudet yoannchaudet merged commit 96a902c into actions:main Jan 31, 2023
@jongwooo jongwooo deleted the chore/use-cache-in-check-dist branch January 31, 2023 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants