Check code with Error Prone 2.7.1 (#480)

Just needed to suppress one new check that crashes on Lombok code. Also added a (non-blocking for now) CI job to test NullAway running on EP 2.7.1.
uber · May 18, 2021 · 4f63460 · 4f63460
1 parent 067c31d
commit 4f63460
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/.github/workflows/continuous-integration.yml b/.github/workflows/continuous-integration.yml
@@ -22,6 +22,9 @@ jobs:
           - os: ubuntu-latest
             java: 8
             epVersion: 2.6.0
+          - os: ubuntu-latest
+            java: 8
+            epVersion: 2.7.1
           - os: ubuntu-latest
             java: 11
             epVersion: 2.4.0

diff --git a/gradle/dependencies.gradle b/gradle/dependencies.gradle
@@ -34,7 +34,7 @@ def versions = [
     asm                    : "7.1",
     checkerFramework       : "3.6.0",
     // The version of Error Prone used to check NullAway's code
-    errorProne             : "2.6.0",
+    errorProne             : "2.7.1",
     // The version of Error Prone that NullAway is compiled and tested against
     errorProneApi          : project.hasProperty("epApiVersion") ? epApiVersion : oldestErrorProneApi,
     support                : "27.1.1",

diff --git a/test-java-lib-lombok/src/main/java/com/uber/lombok/LombokBuilderInit.java b/test-java-lib-lombok/src/main/java/com/uber/lombok/LombokBuilderInit.java
@@ -28,7 +28,10 @@
 
 @Builder
 @Data
-@SuppressWarnings("SameNameButDifferent") // check crashes with EP 2.6.0
+@SuppressWarnings({
+  "SameNameButDifferent" /* crashes with EP 2.6.0 */,
+  "InlineMeInliner" /* crashes with EP 2.7.1 */
+})
 public class LombokBuilderInit {
   private String field;
   @Builder.Default private String fieldWithDefault = "Default";