Use cache action to cache node_modules

[alexandrebodin]

What does it do?

Describe the technical changes you did.

Why is it needed?

Describe the issue you are solving.

How to test it?

Provide information about the environment and the path to verify the behaviour.

Related issue(s)/PR(s)

Let us know if this is related to any issue/pull request

[codecov]

Codecov Report

Base: 59.61% // Head: 59.61% // No change to project coverage 👍

Coverage data is based on head (6658018) compared to base (879c7e2).
Patch has no changes to coverable lines.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #14976   +/-   ##
=======================================
  Coverage   59.61%   59.61%           
=======================================
  Files        1340     1340           
  Lines       32639    32639           
  Branches     6225     6225           
=======================================
  Hits        19459    19459           
  Misses      11316    11316           
  Partials     1864     1864           

Flag	Coverage Δ
back	49.70% <ø> (ø)
front	64.13% <ø> (ø)
unit_back	49.70% <ø> (ø)
unit_front	64.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[joshuaellis]

Do we know that that's the correct cache path?

From previous places i've had to get the cache directory, but then i've normally done this on ubuntu (and the example below uses v2).

[alexandrebodin]

Do we know that that's the correct cache path?

From previous places i've had to get the cache directory, but then i've normally done this on ubuntu (and the example below uses v2).

Your screenshot is for caching yarn cache. Here I'm caching the node_modules directly as we have full control over the node version and can create the cache key based on it.

[joshuaellis]

aha makes sense! 😄 thanks for explaining the diff!

[derrickmehaffy]

Lgtm