-
Notifications
You must be signed in to change notification settings - Fork 80
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
fix: update to setup-gradle #5905
fix: update to setup-gradle #5905
Conversation
This upgrades from the no longer maintained action `burrunan/gradle-cache-action` to the gradle maintained action `gradle/actions/setup-gradle`. There is a different usage pattern; with `setup-gradle`, it is invoked first during setup as opposed to at the command site. We were also using job-ids with `burrunan/gradle-cache-action` to implicitly control cache prefixes; it is not clear if we actually need to separate caches, and so (for now), the default caching behavior of `gradle/actions/setup-gradle` is used, https://github.com/gradle/actions/blob/v3.5.0/docs/setup-gradle.md#caching-build-state-between-jobs. By default, the action will read from cache, but will only write to the cache for jobs on the `main` branch. Fixes deephaven#5161
A nightlies run has succeeded, and I don't see any negative change in build time. |
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.
One version bump, otherwise basically exactly matches my version of the patch.
@@ -29,6 +29,9 @@ jobs: | |||
distribution: 'temurin' | |||
java-version: '17' | |||
|
|||
- name: Setup Gradle | |||
uses: gradle/actions/setup-gradle@v3 |
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.
Looks like these should all be version 4.
uses: gradle/actions/setup-gradle@v3 | |
uses: gradle/actions/setup-gradle@v4 |
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.
v4 is an RC candidate right now, noted in #5898
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.
Hmm okay - their readme suggests v4, so I assumed it was ready for prime time.
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.
Code owner approval
This upgrades from the no longer maintained action
burrunan/gradle-cache-action
to the gradle maintained actiongradle/actions/setup-gradle
. There is a different usage pattern; withsetup-gradle
, it is invoked first during setup as opposed to at the command site. We were also using job-ids withburrunan/gradle-cache-action
to implicitly control cache prefixes; it is not clear if we actually need to separate caches, and so (for now), the default caching behavior ofgradle/actions/setup-gradle
is used, https://github.com/gradle/actions/blob/v3.5.0/docs/setup-gradle.md#caching-build-state-between-jobs. By default, the action will read from cache, but will only write to the cache for jobs on themain
branch.Fixes #5161