Merge branch 'test-8-on-11' into wala-update

nullaway/build.gradle

@@ -96,10 +96,10 @@ test {
 
 apply plugin: 'com.vanniktech.maven.publish'
 
-// Required on Java 11+ since Error Prone and NullAway access a bunch of
-// JDK-internal APIs that are not exposed otherwise
-// MS: the below is unnecessary since we are targeting JDK 8.  Once we target
-// JDK 11, uncomment
+// These --add-exports arguments are required when targeting JDK 11+ since Error Prone and NullAway access a bunch of
+// JDK-internal APIs that are not exposed otherwise.  Since we currently target JDK 8, we do not need to pass the
+// arguments, as encapsulation of JDK internals is not enforced on JDK 8.  In fact, the arguments cause a compiler error
+// when targeting JDK 8.  Leaving commented so we can easily add them back once we target JDK 11.
 //    tasks.withType(JavaCompile).configureEach {
 //        options.compilerArgs += [
 //                "--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED",