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

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
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.

None yet

2 participants