Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Cache Gradle distributions downloaded for a specified version #58

Merged
merged 1 commit into from
Jul 8, 2021

Conversation

bigdaz
Copy link
Member

@bigdaz bigdaz commented Jul 8, 2021

  • Cache is separate from (but similar to) the wrapper distribution cache
  • New 'distributions-cache-enabled' flag controls caching of all downloaded distributions
    (including wrapper distributions)
  • Deprecated the 'wrapper-cache-enabled' flag for removal in v2

- Cache is separate from (but similar to) the wrapper distribution cache
- New 'distributions-cache-enabled' flag controls caching of all downloaded distributions
  (including wrapper distributions)
- Deprecated the 'wrapper-cache-enabled' flag for removal in v2
@bigdaz bigdaz requested a review from eskatos July 8, 2021 14:34
Copy link
Member

@eskatos eskatos left a comment

Choose a reason for hiding this comment

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

💯
Good call to also cache provisioned Gradle distributions!

wrapper-cache-enabled:
description: Whether caching wrapper installation is enabled or not, default to 'true'
required: false
deprecationMessage: Replaced by 'distributions-cache-enabled' which enables caching for all downloaded Gradle distributions
Copy link
Member

Choose a reason for hiding this comment

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

Nice, I didn't know about this features of GitHub Actions 👍

@bigdaz bigdaz merged commit 33e91b6 into master Jul 8, 2021
@bigdaz bigdaz deleted the dd/cache-distributions branch July 8, 2021 18:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants