Skip to content

ci: upgrade actions/cache from v3 to v4 - #565

Merged
paul58914080 merged 1 commit into
mainfrom
ci/codecov
Oct 18, 2025
Merged

ci: upgrade actions/cache from v3 to v4#565
paul58914080 merged 1 commit into
mainfrom
ci/codecov

Conversation

@paul58914080

Copy link
Copy Markdown
Member

Checklist for self review:

@paul58914080
paul58914080 requested a review from a team as a code owner October 18, 2025 04:39
@paul58914080
paul58914080 requested review from Copilot and removed request for a team October 18, 2025 04:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

Upgrade GitHub Actions cache from v3 to v4 in the build workflow to keep CI dependencies current.

  • Bump actions/cache to v4 for Maven dependency caching

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +18 to 21
uses: actions/cache@v4
with:
path: ~/.m2/repository
key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}

Copilot AI Oct 18, 2025

Copy link

Choose a reason for hiding this comment

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

Pin the action to a specific commit SHA instead of the mutable v4 tag to prevent supply-chain risk and unexpected behavior when the tag is updated. Example: uses: actions/cache@ with an inline comment noting the version.

Copilot uses AI. Check for mistakes.
Comment on lines +18 to 21
uses: actions/cache@v4
with:
path: ~/.m2/repository
key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}

Copilot AI Oct 18, 2025

Copy link

Choose a reason for hiding this comment

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

Add restore-keys to improve cache reuse when pom.xml changes, reducing cold builds. For example: restore-keys: | (newline) ${{ runner.os }}-maven-

Copilot uses AI. Check for mistakes.
Comment on lines 15 to 17
distribution: temurin
java-version: 21
- name: Cache Maven dependencies

Copilot AI Oct 18, 2025

Copy link

Choose a reason for hiding this comment

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

actions/setup-java supports built-in Maven caching (cache: maven). Enabling it on the setup-java step can simplify the workflow and eliminate the separate actions/cache step.

Copilot uses AI. Check for mistakes.
@codecov

codecov Bot commented Oct 18, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.22%. Comparing base (83dfcd7) to head (0e4db79).
⚠️ Report is 78 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #565      +/-   ##
============================================
+ Coverage     87.69%   89.22%   +1.53%     
- Complexity      188      204      +16     
============================================
  Files            51       51              
  Lines           520      594      +74     
  Branches         21       21              
============================================
+ Hits            456      530      +74     
  Misses           53       53              
  Partials         11       11              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@paul58914080
paul58914080 merged commit 5ac7a12 into main Oct 18, 2025
4 checks passed
@paul58914080
paul58914080 deleted the ci/codecov branch October 18, 2025 04:48
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.

2 participants