Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimized Github Actions cache for dependency updates #455

Merged
merged 1 commit into from Mar 27, 2022

Conversation

freemanjp
Copy link
Member

Changed to cache by date interval rather than file hash. Most commits to this project are dependency updates, which results in a poor cache hit rate. Better that most of the dependencies are cached than none. Also, better to avoid updating the cache on every commit given most changes are only affect a single dependency.

@freemanjp freemanjp force-pushed the cache-by-date-interval branch 6 times, most recently from 731b5d1 to 104d6f5 Compare March 27, 2022 20:50
Changed to cache by date interval rather than file hash. Most commits to this project are dependency updates, which results in a poor cache hit rate. Better that most of the dependencies are cached than none. Also, better to avoid updating the cache on every commit given most changes are only affect a single dependency.
@codecov
Copy link

codecov bot commented Mar 27, 2022

Codecov Report

Merging #455 (0d8ad6f) into master (5748e1f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #455   +/-   ##
=========================================
  Coverage     85.34%   85.34%           
  Complexity       91       91           
=========================================
  Files            16       16           
  Lines           539      539           
  Branches         71       71           
=========================================
  Hits            460      460           
  Misses           34       34           
  Partials         45       45           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5748e1f...0d8ad6f. Read the comment docs.

@freemanjp freemanjp merged commit d64e7c5 into master Mar 27, 2022
@freemanjp freemanjp deleted the cache-by-date-interval branch March 27, 2022 21:01
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.

None yet

1 participant