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

Jump to bottom

Cache-tuning parameters should be multi-line string inputs #106

Closed
bigdaz opened this issue Oct 29, 2021 · 0 comments
Closed

Cache-tuning parameters should be multi-line string inputs #106

bigdaz opened this issue Oct 29, 2021 · 0 comments
Assignees
@bigdaz
Labels
enhancement New feature or request
Milestone
v2.0

Comments

@bigdaz
Copy link
Member

bigdaz commented Oct 29, 2021

The gradle-home-cache-includes and gradle-home-cache-excludes parameters were initially implemented as JSON string inputs. This makes these inputs non-idiomatic and easier to get wrong.
Since the only permissible input is an array of Strings, we should convert these to multi-line input parameters.
@bigdaz bigdaz added the enhancement New feature or request label Oct 29, 2021
@bigdaz bigdaz added this to the v2.0 milestone Oct 29, 2021
@bigdaz bigdaz self-assigned this Oct 29, 2021
@bigdaz bigdaz mentioned this issue Oct 30, 2021
Merged
@bigdaz bigdaz closed this as completed in e3ada7e Oct 30, 2021
bigdaz added a commit that referenced this issue Oct 30, 2021
@bigdaz
Merge pull request #109 from gradle/dd/rc21
613f4ec 
Fix issues for 2.0-rc.2
- Support multi-line strings for cache-tuning parameters #106 
- Include all downloaded files in `dependencies` bundle #100 
- Only restore configuration-cache if Gradle User Home is fully restored #107
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@bigdaz bigdaz

Labels
enhancement New feature or request
Projects
None yet
Milestone
v2.0
Development

No branches or pull requests
1 participant
@bigdaz