-
Notifications
You must be signed in to change notification settings - Fork 4
AAE-26175 Fix on mvn build cache keys #1164
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
AAE-26175 Fix on mvn build cache keys #1164
Conversation
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.
Pull Request Overview
This PR fixes Maven build cache keys to improve caching behavior and adds missing Maven credentials to JaCoCo steps. The version is bumped from v9.2.0 to v9.2.1 as a patch release.
- Updated Maven cache restore keys to remove trailing dashes and use consistent PR number references
- Added Maven username/password environment variables to JaCoCo merge and report steps
- Minor formatting cleanup by removing an empty line
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
version.txt | Bumped version from v9.2.0 to v9.2.1 for patch release |
.github/actions/maven-configure/action.yml | Fixed cache restore keys by removing trailing dashes and updating PR number reference |
.github/actions/maven-build/action.yml | Added Maven credentials to JaCoCo merge and report steps |
.github/actions/maven-build-and-tag/action.yml | Removed empty line for formatting cleanup |
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.
please address use case where GITHUB_PR_NUMBER is not filled in env (see copilot's review)
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.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
…rove cache save conditions
…lity and refine cache save conditions
Checklist
Description