Use CompileStatic

src/main/groovy/com/netflix/nebula/config/plugin/ConfigurationEnvironmentPlugin.groovy

@@ -1,5 +1,6 @@
 package com.netflix.nebula.config.plugin
 
+import groovy.transform.CompileStatic
 import org.gradle.api.Plugin
 import org.gradle.api.Project
 
@@ -9,6 +10,7 @@ import org.gradle.api.Project
  * new lint rules that may depend on manipulating dependencies in a
  * configuration-dependent way.
  */
+@CompileStatic
 class ConfigurationEnvironmentPlugin implements Plugin<Project> {
     @Override
     void apply(Project project) {

src/main/groovy/com/netflix/nebula/lint/FileMode.groovy

@@ -16,8 +16,11 @@
 
 package com.netflix.nebula.lint
 
+import groovy.transform.CompileStatic
+
 import static java.nio.file.Files.isSymbolicLink
 
+@CompileStatic
 enum FileMode {
     Regular(100644),
     Symlink(120000),

src/main/groovy/com/netflix/nebula/lint/GradleLintFix.groovy

@@ -16,6 +16,8 @@
 
 package com.netflix.nebula.lint
 
+import groovy.transform.CompileStatic
+
 import java.nio.file.Files
 
 /**
@@ -59,6 +61,7 @@ abstract class GradleLintMultilineFix extends GradleLintFix {
     Integer to() { affectedLines.to }
 }
 
+@CompileStatic
 class GradleLintReplaceWith extends GradleLintMultilineFix {
     Integer fromColumn // the first affected column of the first line (1-based, inclusive)
     Integer toColumn // the last affected column of the last line (1-based, exclusive)
@@ -78,6 +81,7 @@ class GradleLintReplaceWith extends GradleLintMultilineFix {
     String changes() { changes }
 }
 
+@CompileStatic
 class GradleLintDeleteLines extends GradleLintMultilineFix {
 
     GradleLintDeleteLines(GradleViolation violation, File affectedFile, Range<Integer> affectedLines) {
@@ -90,6 +94,7 @@ class GradleLintDeleteLines extends GradleLintMultilineFix {
     String changes() { null }
 }
 
+@CompileStatic
 class GradleLintInsertAfter extends GradleLintFix {
     Integer afterLine // 1-based
     String changes
@@ -111,6 +116,7 @@ class GradleLintInsertAfter extends GradleLintFix {
     String changes() { changes }
 }
 
+@CompileStatic
 class GradleLintInsertBefore extends GradleLintFix {
     Integer beforeLine // 1-based
     String changes
@@ -132,6 +138,7 @@ class GradleLintInsertBefore extends GradleLintFix {
     String changes() { changes }
 }
 
+@CompileStatic
 class GradleLintDeleteFile extends GradleLintMultilineFix implements RequiresOwnPatchset {
     GradleLintDeleteFile(GradleViolation violation, File affectedFile) {
         this.violation = violation
@@ -144,6 +151,7 @@ class GradleLintDeleteFile extends GradleLintMultilineFix implements RequiresOwn
     String changes() { null }
 }
 
+@CompileStatic
 class GradleLintCreateFile extends GradleLintInsertBefore implements RequiresOwnPatchset {
     FileMode fileMode
 

src/main/groovy/com/netflix/nebula/lint/GradleViolation.groovy

@@ -17,6 +17,7 @@
 package com.netflix.nebula.lint
 
 import com.netflix.nebula.lint.rule.BuildFiles
+import groovy.transform.CompileStatic
 import groovy.transform.EqualsAndHashCode
 import org.codehaus.groovy.ast.ASTNode
 import org.codehaus.groovy.ast.expr.ArgumentListExpression
@@ -27,6 +28,7 @@ import org.codenarc.rule.Violation
 
 import java.util.concurrent.atomic.AtomicInteger
 
+@CompileStatic
 @EqualsAndHashCode(includes = 'id')
 class GradleViolation extends Violation {
     BuildFiles files

src/main/groovy/com/netflix/nebula/lint/UnfixedViolationReason.groovy

@@ -16,6 +16,9 @@
 
 package com.netflix.nebula.lint
 
+import groovy.transform.CompileStatic
+
+@CompileStatic
 enum UnfixedViolationReason {
     OverlappingPatch('one or more fixes overlap with another, run fixGradleLint again to apply the change')
 

src/main/groovy/com/netflix/nebula/lint/rule/BuildFiles.groovy

@@ -1,9 +1,12 @@
 package com.netflix.nebula.lint.rule
 
+import groovy.transform.CompileStatic
+
 /**
  * This class provides transformation from multiple build files to one concatenated text which is used for applying
  * lint rules. We keep original mapping so we can get based on line number in concatenated text original line and file.
  */
+@CompileStatic
 class BuildFiles {
 
     private Map<LineRange, File> orderedFiles = new LinkedHashMap<>()
@@ -53,7 +56,7 @@ class BuildFiles {
         } else {
             new Original(
                     file: orderedFiles.values().first(),
-                    line: null
+                    line: null as Integer
             )
         }
     }

src/main/groovy/com/netflix/nebula/lint/rule/FixmeRule.groovy

@@ -1,9 +1,11 @@
 package com.netflix.nebula.lint.rule
 
+import groovy.transform.CompileStatic
 import org.codenarc.rule.Rule
 import org.codenarc.rule.Violation
 import org.codenarc.source.SourceCode
 
+@CompileStatic
 class FixmeRule implements Rule {
     @Override
     List<Violation> applyTo(SourceCode sourceCode) {

src/main/groovy/com/netflix/nebula/lint/rule/GradleDependency.groovy

@@ -17,11 +17,14 @@
 package com.netflix.nebula.lint.rule
 
 import groovy.transform.Canonical
+import groovy.transform.CompileDynamic
+import groovy.transform.CompileStatic
 import org.gradle.api.artifacts.ModuleIdentifier
 import org.gradle.api.artifacts.ModuleVersionIdentifier
 import org.gradle.api.internal.artifacts.DefaultModuleIdentifier
 import org.gradle.api.internal.artifacts.DefaultModuleVersionIdentifier
 
+@CompileStatic
 @Canonical
 class GradleDependency implements Cloneable {
     String group
@@ -38,10 +41,12 @@ class GradleDependency implements Cloneable {
         EvaluatedArbitraryCode
     }
 
+    @CompileDynamic
     ModuleVersionIdentifier toModuleVersion() {
         return new DefaultModuleVersionIdentifier(group, name, version)
     }
-
+
+    @CompileDynamic
     ModuleIdentifier toModule() {
         return new DefaultModuleIdentifier(group, name)
     }

src/main/groovy/com/netflix/nebula/lint/rule/GradlePlugin.groovy

@@ -1,8 +1,10 @@
 package com.netflix.nebula.lint.rule
 
 import groovy.transform.Canonical
+import groovy.transform.CompileStatic
 
 @Canonical
+@CompileStatic
 class GradlePlugin {
     String id
 }

src/main/groovy/com/netflix/nebula/lint/rule/dependency/AbstractDuplicateDependencyClassRule.groovy

@@ -3,18 +3,21 @@ package com.netflix.nebula.lint.rule.dependency
 import com.netflix.nebula.lint.rule.GradleDependency
 import com.netflix.nebula.lint.rule.GradleLintRule
 import com.netflix.nebula.lint.rule.GradleModelAware
+import groovy.transform.CompileDynamic
+import groovy.transform.CompileStatic
 import org.codehaus.groovy.ast.ClassNode
 import org.codehaus.groovy.ast.expr.MethodCallExpression
 import org.gradle.api.artifacts.Configuration
 import org.gradle.api.artifacts.ModuleVersionIdentifier
 
+@CompileStatic
 abstract class AbstractDuplicateDependencyClassRule extends GradleLintRule implements GradleModelAware {
     String description = 'classpaths with duplicate classes may break unpredictably depending on the order in which dependencies are provided to the classpath'
 
     Set<Configuration> directlyUsedConfigurations = [] as Set
     Set<ModuleVersionIdentifier> ignoredDependencies = [] as Set
 
-    def resolvableAndResolvedConfigurations
+    Set<Configuration> resolvableAndResolvedConfigurations
 
     abstract protected List<ModuleVersionIdentifier> moduleIds(Configuration conf)
 

src/main/groovy/com/netflix/nebula/lint/rule/dependency/ClassHierarchyUtils.groovy

@@ -16,10 +16,12 @@
 
 package com.netflix.nebula.lint.rule.dependency
 
+import groovy.transform.CompileStatic
 import groovy.transform.Memoized
 import org.slf4j.Logger
 import org.slf4j.LoggerFactory
 
+@CompileStatic
 class ClassHierarchyUtils {
     private static Logger logger = LoggerFactory.getLogger(ClassHierarchyUtils)
 

src/main/groovy/com/netflix/nebula/lint/rule/dependency/ClassInformation.groovy

@@ -1,5 +1,8 @@
 package com.netflix.nebula.lint.rule.dependency
 
+import groovy.transform.CompileStatic
+
+@CompileStatic
 class ClassInformation {
     String filePath
     String source

src/main/groovy/com/netflix/nebula/lint/rule/dependency/DependencyTupleExpressionRule.groovy

@@ -20,8 +20,10 @@ import com.netflix.nebula.lint.rule.GradleAstUtil
 import com.netflix.nebula.lint.rule.GradleDependency
 import com.netflix.nebula.lint.rule.GradleLintRule
 import com.netflix.nebula.lint.rule.GradleModelAware
+import groovy.transform.CompileStatic
 import org.codehaus.groovy.ast.expr.MethodCallExpression
 
+@CompileStatic
 class DependencyTupleExpressionRule extends GradleLintRule implements GradleModelAware {
     String description = "use the more compact string representation of a dependency when possible"
 

src/main/groovy/com/netflix/nebula/lint/rule/dependency/DependencyViolationUtil.groovy

@@ -2,8 +2,10 @@ package com.netflix.nebula.lint.rule.dependency
 
 import com.netflix.nebula.lint.GradleViolation
 import com.netflix.nebula.lint.rule.GradleDependency
+import groovy.transform.CompileStatic
 import org.codehaus.groovy.ast.expr.MethodCallExpression
 
+@CompileStatic
 class DependencyViolationUtil {
 
     static void replaceProjectDependencyConfiguration(GradleViolation violation, MethodCallExpression call, String configuration, String project) {

src/main/groovy/com/netflix/nebula/lint/rule/dependency/DeprecatedDependencyConfigurationRule.groovy

@@ -5,8 +5,11 @@ import com.netflix.nebula.lint.GradleViolation
 import com.netflix.nebula.lint.rule.GradleDependency
 import com.netflix.nebula.lint.rule.GradleLintRule
 import com.netflix.nebula.lint.rule.GradleModelAware
+import groovy.transform.CompileDynamic
+import groovy.transform.CompileStatic
 import org.codehaus.groovy.ast.expr.MethodCallExpression
 
+@CompileStatic
 class DeprecatedDependencyConfigurationRule extends GradleLintRule implements GradleModelAware {
     String description = 'Replace deprecated configurations in dependencies'
 
@@ -51,6 +54,7 @@ class DeprecatedDependencyConfigurationRule extends GradleLintRule implements Gr
      * Responsible for replacing configurations in project dependencies only
      * @param call
      */
+    @CompileDynamic
     private void handleProjectDependencies(MethodCallExpression call) {
         List statements = call.arguments.expressions*.code*.statements.flatten().findAll  { it.expression instanceof MethodCallExpression }
         statements.each { statement ->
@@ -71,6 +75,7 @@ class DeprecatedDependencyConfigurationRule extends GradleLintRule implements Gr
         }
     }
 
+    @CompileDynamic
     private void handleDependencyVisit(MethodCallExpression call, String conf, GradleDependency dep) {
         if(CONFIGURATION_REPLACEMENTS.containsKey(conf) && !GradleKt.versionLessThan(project.gradle, MINIMUM_GRADLE_VERSION)) {
             if (call.arguments.expressions.size() == 1) {

src/main/groovy/com/netflix/nebula/lint/rule/dependency/FirstOrderDuplicateDependencyClassRule.groovy

@@ -1,8 +1,10 @@
 package com.netflix.nebula.lint.rule.dependency
 
+import groovy.transform.CompileStatic
 import org.gradle.api.artifacts.Configuration
 import org.gradle.api.artifacts.ModuleVersionIdentifier
 
+@CompileStatic
 class FirstOrderDuplicateDependencyClassRule extends AbstractDuplicateDependencyClassRule {
     @Override
     protected List<ModuleVersionIdentifier> moduleIds(Configuration conf) {

src/main/groovy/com/netflix/nebula/lint/rule/dependency/JarContents.groovy

@@ -1,5 +1,8 @@
 package com.netflix.nebula.lint.rule.dependency
 
+import groovy.transform.CompileStatic
+
+@CompileStatic
 class JarContents {
     Set<String> entryNames
 

src/main/groovy/com/netflix/nebula/lint/rule/dependency/MethodReference.groovy

@@ -1,6 +1,9 @@
 package com.netflix.nebula.lint.rule.dependency
 
+import groovy.transform.CompileStatic
 
+
+@CompileStatic
 class MethodReference {
     String methodName
     String owner
@@ -32,13 +35,13 @@ class MethodReference {
         INVOKEINTERFACE(185),
         INVOKEDYNAMIC(186)
 
-        private int code
+        int code
 
         OpCode(int code) {
             this.code = code
         }
 
-        static findByCode(int code) {
+        static OpCode findByCode(int code) {
             values().find { it.code == code }
         }
     }

src/main/groovy/com/netflix/nebula/lint/rule/dependency/MethodScanner.groovy

@@ -16,7 +16,7 @@ import java.nio.file.Path
 class MethodScanner {
     private AppClassVisitor classVisitor
 
-    private ArrayList<MethodReference> methodReferences = []
+    private List<MethodReference> methodReferences = []
 
     private class AppMethodVisitor extends MethodVisitor {
 

src/main/groovy/com/netflix/nebula/lint/rule/dependency/MinimumDependencyVersionRule.groovy

@@ -3,6 +3,7 @@ package com.netflix.nebula.lint.rule.dependency
 import com.netflix.nebula.lint.rule.GradleDependency
 import com.netflix.nebula.lint.rule.GradleLintRule
 import com.netflix.nebula.lint.rule.GradleModelAware
+import groovy.transform.CompileStatic
 import org.codehaus.groovy.ast.ASTNode
 import org.codehaus.groovy.ast.ClassNode
 import org.codehaus.groovy.ast.expr.Expression
@@ -18,10 +19,11 @@ import org.gradle.util.VersionNumber
  * Continuing confluence of these two plugins may eventually result in a different way to parameterize the rule.
  */
 @Incubating
+@CompileStatic
 class MinimumDependencyVersionRule extends GradleLintRule implements GradleModelAware {
     String description = 'pull up dependencies to a minimum version if necessary'
-    def alreadyAdded = [] as Set
-    def resolvableAndResolvedConfigurations
+    Set alreadyAdded = [] as Set
+    Set<Configuration> resolvableAndResolvedConfigurations
 
     @Lazy
     List<GradleDependency> minimumVersions = {
@@ -32,7 +34,7 @@ class MinimumDependencyVersionRule extends GradleLintRule implements GradleModel
                     collect { GradleDependency.fromConstant(it) }?.
                     findAll { it != null } ?:
                     []
-        } else []
+        } else [] as List<GradleDependency>
     }()
 
     @Override

src/main/groovy/com/netflix/nebula/lint/rule/dependency/ModuleDescriptor.groovy

@@ -19,9 +19,11 @@
 package com.netflix.nebula.lint.rule.dependency
 
 import com.netflix.nebula.lint.rule.GradleDependency
+import groovy.transform.CompileStatic
 import groovy.transform.Immutable
 
 @Immutable
+@CompileStatic
 class ModuleDescriptor {
     String group
     String name