Skip to content

Smarter Caching on Github Actions #2788

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

Merged
merged 2 commits into from
Jun 13, 2020

Conversation

cortinico
Copy link
Member

Currently our Github Actions setup relies on a static key for caching.

This means that we restore an old Gradle cache that doesn't contain the version of Gradle we need (it gets re-downloaded every time).

Given that we don't have a lockfile (should we consider it?), I'm hashing 'buildSrc/src/main/kotlin/Versions.kt' and adding it to the cache key. So the cache will be partially invalidated whenever we release a new version (or whenever we bump a library).

@codecov
Copy link

codecov bot commented Jun 11, 2020

Codecov Report

Merging #2788 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2788   +/-   ##
=========================================
  Coverage     80.51%   80.51%           
  Complexity     2324     2324           
=========================================
  Files           386      386           
  Lines          6955     6955           
  Branches       1262     1262           
=========================================
  Hits           5600     5600           
  Misses          725      725           
  Partials        630      630           

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 52e827d...2a9c55e. Read the comment docs.

Copy link
Member

@arturbosch arturbosch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 so lets wait for the bom PR then

@arturbosch arturbosch added this to the 1.10.0 milestone Jun 13, 2020
@arturbosch arturbosch added the housekeeping Marker for housekeeping tasks and refactorings label Jun 13, 2020
@cortinico cortinico force-pushed the nc/actions-smarter-caching branch from 00ebf8b to 3bb2890 Compare June 13, 2020 11:51
@cortinico
Copy link
Member Author

CI is red because of #2795

@cortinico cortinico force-pushed the nc/actions-smarter-caching branch from 3bb2890 to 2a9c55e Compare June 13, 2020 12:52
@schalkms schalkms merged commit 77fa5f0 into detekt:master Jun 13, 2020
@cortinico cortinico deleted the nc/actions-smarter-caching branch June 13, 2020 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Marker for housekeeping tasks and refactorings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants