Skip to content
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 gradle/gradle-build-action #6121

Merged
merged 2 commits into from Nov 4, 2022
Merged

Use gradle/gradle-build-action #6121

merged 2 commits into from Nov 4, 2022

Conversation

eddumelendez
Copy link
Member

@eddumelendez eddumelendez commented Nov 3, 2022

Those are the benefits we can get from the action and our current usage:

  • Efficient cache
  • Summary report
  • Capture of build scan links

Those are the enefits we can get from the action and our current usage:
* Efficient cache
* Summary report
* Capture of build scan links
@eddumelendez eddumelendez requested a review from a team as a code owner November 3, 2022 17:49
@github-actions github-actions bot added the github_actions Pull requests that update Github_actions code label Nov 3, 2022
@eddumelendez
Copy link
Member Author

See summary

- name: Setup Gradle Build Action
uses: gradle/gradle-build-action@v2
with:
gradle-home-cache-cleanup: true
Copy link
Member

Choose a reason for hiding this comment

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

I had to look up the effect of it because I think the name is confusing but it looks like a good idea. I wonder why it is not a default?

https://github.com/gradle/gradle-build-action#removing-unused-files-from-gradle-user-home-before-saving-to-cache

Copy link
Member Author

Choose a reason for hiding this comment

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

it is experimental but what can go wrong? 🤷🏽‍♂️

Comment on lines +26 to 31
- name: Setup Gradle Build Action
uses: gradle/gradle-build-action@v2

- name: Run Gradle Build
run: ./gradlew build --scan --no-daemon -i -x test

Copy link
Member

Choose a reason for hiding this comment

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

We can also switch to using the gradle-build-action for execution, but it probably changes nothing.

https://github.com/gradle/gradle-build-action#gradle-execution

Copy link
Member Author

Choose a reason for hiding this comment

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

that's useful when testing with different gradle versions. In our case would not help that much given we use gradle wrapper

@kiview kiview added this to the next milestone Nov 4, 2022
@kiview
Copy link
Member

kiview commented Nov 4, 2022

The summary report is a great feature and it makes the build scans very accessible 👍

@kiview kiview merged commit de20484 into main Nov 4, 2022
@kiview kiview deleted the build-gradle-action branch November 4, 2022 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update Github_actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants