From 052ff060b18491d178a68336a68b4f2b959d2de2 Mon Sep 17 00:00:00 2001
From: Harsh Shandilya <me@msfjarvis.dev>
Date: Wed, 22 Mar 2023 21:27:50 +0530
Subject: [PATCH] gradle: use idiomatic task handling

- The hand-rolled clean task is not required

- Tasks should use configureEach to prevent eager evaluation

Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
---
 build.gradle    |  4 ----
 ui/build.gradle | 10 +++++-----
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/build.gradle b/build.gradle
index 46d94a33..448162f0 100644
--- a/build.gradle
+++ b/build.gradle
@@ -23,10 +23,6 @@ buildscript {
     }
 }
 
-task clean(type: Delete) {
-    delete rootProject.buildDir
-}
-
 tasks {
     wrapper {
         gradleVersion = "8.0.2"
diff --git a/ui/build.gradle b/ui/build.gradle
index a5e7b4fa..2c2a3fa8 100644
--- a/ui/build.gradle
+++ b/ui/build.gradle
@@ -1,3 +1,5 @@
+import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
+
 plugins {
     id 'com.android.application'
     id 'org.jetbrains.kotlin.android'
@@ -82,13 +84,11 @@ dependencies {
     coreLibraryDesugaring "com.android.tools:desugar_jdk_libs:$desugarVersion"
 }
 
-tasks.withType(JavaCompile) {
+tasks.withType(JavaCompile).configureEach {
     options.compilerArgs << '-Xlint:unchecked'
     options.deprecation = true
 }
 
-tasks.withType(org.jetbrains.kotlin.gradle.tasks.KotlinCompile).all {
-    kotlinOptions {
-        jvmTarget = JavaVersion.VERSION_1_8
-    }
+tasks.withType(KotlinCompile).configureEach {
+    kotlinOptions.jvmTarget = JavaVersion.VERSION_1_8
 }