[beam] branch master updated: [BEAM-11837] Make checker framework opt-out rather than opt-in.

[ibzib]

This is an automated email from the ASF dual-hosted git repository.

ibzib pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new 78721ef  [BEAM-11837] Make checker framework opt-out rather than 
opt-in.
     new 88fc2ad  Merge pull request #14301 from ibzib/BEAM-11837
78721ef is described below

commit 78721ef6ee9bab994176710dd6666396ca578f24
Author: Kyle Weaver <kcwea...@google.com>
AuthorDate: Mon Mar 22 15:03:49 2021 -0700

    [BEAM-11837] Make checker framework opt-out rather than opt-in.
    
    You can opt in by passing -PenableCheckerFramework=true to the Gradle 
invocation.
---
 .../src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy  | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy 
b/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
index 939851c..d63a594 100644
--- a/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
+++ b/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
@@ -836,6 +836,12 @@ class BeamModulePlugin implements Plugin<Project> {
           'org.checkerframework.checker.nullness.NullnessChecker'
         ]
 
+        if (project.findProperty('enableCheckerFramework') || 
project.jenkins.isCIBuild) {
+          skipCheckerFramework = false
+        } else {
+          skipCheckerFramework = true
+        }
+
         // Always exclude checkerframework on tests. It's slow, and it often
         // raises erroneous error because we don't have checker annotations for
         // test libraries like junit and hamcrest. See BEAM-11436.