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

Support configuration cache #291

Merged
merged 2 commits into from
Apr 23, 2022

Conversation

MyDogTom
Copy link
Contributor

@MyDogTom MyDogTom commented Feb 27, 2022

📌 References

🎩 What is the goal?

Support Gradle configuration cache.

How is it being implemented?

  • Avoid using project during task execution. Add extra parameters instead.
  • Don't use BuildType as parameter. Use directly buildTypeName as parameter instead.

How can it be tested?

Ideally, I could enable configuration cache in shot-consumer project, but this requires to update AGP and Gradle. Should I do the update? Do you prefer that I do it as current PR or as separate PR? I see that there is a pending PR with an update. #288

Disclaimer

I don't know Scala. All changes were done just by following current code. Please keep that in mind during code review.

PS. BTW, having Kotlin as programming language would enable much more people to contribute.

* Avoid using project during task execution. Add extra parameters instead.
* Don't use BuildType as parameter. Use directly buildTypeName as parameter instead.
@MyDogTom MyDogTom marked this pull request as ready for review February 27, 2022 21:00
Copy link
Owner

@pedrovgs pedrovgs left a comment

Choose a reason for hiding this comment

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

Thank you so much for this contribution 👏 I'll release a new version with this tomorrow 😃

@pedrovgs pedrovgs merged commit a5fe2e4 into pedrovgs:master Apr 23, 2022
@MyDogTom MyDogTom deleted the support-configuration-cache branch April 24, 2022 16:08
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.

Gradle Configuration Cache compatibility
2 participants