CirclesUBI / circles-myxogastria

Webapp and mobile client for Circles
https://circles.garden
GNU Affero General Public License v3.0
37 stars 21 forks source link

Update actions versions and use setup-node for caching #656

Closed llunaCreixent closed 1 year ago

llunaCreixent commented 1 year ago

Using the .nvm file works. However, caching the dependencies is not working. I suggest keep using the actions/cache action.

netlify[bot] commented 1 year ago

Deploy Preview for relaxed-snyder-612ce8 ready!

Name Link
Latest commit f1593a97b44b29188188721009ec685e28db4a61
Latest deploy log https://app.netlify.com/sites/relaxed-snyder-612ce8/deploys/6436a08c560a5700087a8481
Deploy Preview https://deploy-preview-656--relaxed-snyder-612ce8.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

llunaCreixent commented 1 year ago

Check in the Action the following log: npm cache is not found

llunaCreixent commented 1 year ago

Do you know how to cache the dependencies without using the actions/cache action? Here it says it's not necessary since you can configure the cache with setup-node action. We have 2 options:

  1. Keep using actions/cache
  2. Learns how to configure the cache through setup-node action
llunaCreixent commented 1 year ago

Perfect! We accomplished option 2 (configure the cache through setup-node action). You can check how the cache is found here: https://github.com/CirclesUBI/circles-myxogastria/actions/runs/4678146854/jobs/8286524579

All looks good can be reused for other repos