Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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