Include cache hit in example

[JJ]

cache-hit is in fact not mentioned anywhere in the README file

Description:

Adds the use of cache-hit to the example.

Related issue:

It's so simple the PR is possibly the best issue.

Check list:

• Mark if documentation changes are required.
• Mark if tests were added or updated to cover the changes.

[dmitry-shibanov]

Hello @JJ. Thank you for your pull request. The example won't work because the action caches global directory, but not node_modules.

[JJ]

Hello @JJ. Thank you for your pull request. The example won't work because the action caches global directory, but not node_modules.

That would explain a lot of things. But then what it does is to download the dependent modules?
Is it possible to salvage this PR by clarifying exactly what the cache does and where does the speedups will be enjoyed?

[dmitry-shibanov]

Hello @JJ. Thank you for your answer. Actually we have an information about cache in the advanced section. The action caches global cache directory. For example we use the npm get config cache to get global cache directory for npm.

[dmitry-shibanov]

Hello @JJ. For now I'm going to close the pull request. Feel free to contact us If you have any questions.