Merge pull request #17195 from wordpress-mobile/analysis/annotations-all-warnings-as-errors

[Compile Warnings As Errors] Annotations Module - Enable All Warnings as Errors

Author: ParaskP7

build.gradle

@@ -1,3 +1,5 @@
+import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
+
 plugins {
     id "com.automattic.android.fetchstyle"
     id "io.gitlab.arturbosch.detekt"
@@ -103,9 +105,9 @@ allprojects {
         debug = false
     }
 
-    tasks.withType(org.jetbrains.kotlin.gradle.tasks.KotlinCompile).all {
+    tasks.withType(KotlinCompile).all {
         kotlinOptions {
-            jvmTarget = "1.8"
+            jvmTarget = JavaVersion.VERSION_1_8
         }
     }
 }

libs/annotations/build.gradle

@@ -2,19 +2,8 @@ plugins {
     id "org.jetbrains.kotlin.jvm"
 }
 
-sourceCompatibility = "7"
-targetCompatibility = "7"
-
-repositories {
-    mavenCentral()
-}
 compileKotlin {
     kotlinOptions {
-        jvmTarget = "1.8"
-    }
-}
-compileTestKotlin {
-    kotlinOptions {
-        jvmTarget = "1.8"
+        allWarningsAsErrors = true
     }
 }

libs/editor/build.gradle

@@ -41,10 +41,6 @@ android {
         minSdkVersion rootProject.minSdkVersion
         targetSdkVersion rootProject.targetSdkVersion
     }
-    compileOptions {
-        sourceCompatibility 1.8
-        targetCompatibility 1.8
-    }
 
     // Avoid 'duplicate files during packaging of APK' errors
     packagingOptions {

libs/image-editor/build.gradle

@@ -18,10 +18,6 @@ android {
         vectorDrawables.useSupportLibrary = true
     }
 
-    kotlinOptions {
-        jvmTarget = JavaVersion.VERSION_1_8
-    }
-
     sourceSets {
         main.java.srcDirs += 'src/main/kotlin'
         test.java.srcDirs += 'src/test/kotlin'

libs/processors/build.gradle

@@ -17,7 +17,3 @@ dependencies {
     testImplementation "com.nhaarman.mockitokotlin2:mockito-kotlin:$nhaarmanMockitoVersion"
     testImplementation "org.assertj:assertj-core:$assertJVersion"
 }
-
-sourceCompatibility = "7"
-targetCompatibility = "7"
-