kou commented on PR #36373:
URL: https://github.com/apache/arrow/pull/36373#issuecomment-1616826627

   Then, we can use `js-jest-cache-${{ runner.os }}-${{ hashFiles('js/**/*.ts', 
''js/test/**/*.ts') }}` for `key` like other our workflow files. If we can't 
reuse most cache when we update dependencies, we can also add `js/yarn.lock` 
like `js-jest-cache-${{ runner.os }}-${{ hashFiles('js/**/*.ts', 
''js/test/**/*.ts', 'js/yarn.lock') }}`.
   
   We should not use `github.sha`. If we use it, our cache is always updated 
even when we don't change `js/{src,test}/**/*.ts`. It will waste our available 
cache limit.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to