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
Use the setup-java caching #698
Conversation
The caching uses actions/cache@v2 under-the-hood. This reduces our YAML debt.
- uses: actions/checkout@v2 | ||
- uses: actions/setup-java@v2 | ||
name: set up jdk ${{matrix.java.version}} | ||
with: | ||
distribution: temurin | ||
java-version: ${{matrix.java.version}} | ||
cache: maven |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It only works because we still compile our artifacts for Java 8, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It caches the dependencies, not the produced artefacts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's actually the same as the old caching, except that the setup-java
action nows manages the caching one.
Codecov Report
@@ Coverage Diff @@
## main #698 +/- ##
============================================
+ Coverage 90.07% 90.14% +0.06%
Complexity 2993 2993
============================================
Files 374 374
Lines 11791 11791
Branches 1480 1480
============================================
+ Hits 10621 10629 +8
+ Misses 600 594 -6
+ Partials 570 568 -2
|
The caching uses actions/cache@v2 under-the-hood.
This reduces our YAML debt.