Skip to content

Commit 8f68736

Browse files
committed
Re-enable errorprone, turn off EffectivelyPrivate check. Log all enabled checks. Follow-up to #15305.
1 parent 42020b1 commit 8f68736

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

build-tools/build-infra/src/main/java/org/apache/lucene/gradle/plugins/java/ErrorPronePlugin.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,7 @@ public void apply(Project project) {
379379
"DurationGetTemporalUnit:OFF", // we don't use Duration.get()
380380
"DurationTemporalUnit:OFF", // we don't use Duration.of() etc
381381
"DurationToLongTimeUnit:OFF", // we don't use TimeUnit.convert Duration, etc
382-
"EffectivelyPrivate:ERROR", // TODO: many violations of this.
382+
"EffectivelyPrivate:OFF", // TODO: many violations of this.
383383
"EmptyBlockTag:OFF", // ECJ takes care
384384
"EmptyCatch:OFF", // ECJ takes care
385385
"EmptyIf:OFF", // TODO: new, not checked if applicable to Lucene
@@ -881,7 +881,7 @@ public void apply(Project project) {
881881
/** Filter out all checks which are disabled. */
882882
private static final Map<String, CheckSeverity> ENABLED_CHECKS =
883883
ALL_CHECKS_PARSED.entrySet().stream()
884-
.filter(e -> e.getValue() == CheckSeverity.OFF)
884+
.filter(e -> e.getValue() != CheckSeverity.OFF)
885885
.collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
886886

887887
private void configureErrorProne(Project project) {
@@ -911,6 +911,10 @@ private void configureErrorProne(Project project) {
911911
// Turn off everything by default, then re-enable what's needed.
912912
epOptions.getDisableAllChecks().set(true);
913913
epOptions.getChecks().putAll(ENABLED_CHECKS);
914+
t.getLogger()
915+
.info(
916+
"Compiling with error prone, enabled checks: "
917+
+ String.join(", ", ENABLED_CHECKS.keySet()));
914918

915919
// Exclude certain files (generated ones, mostly).
916920
List<String> excludedPaths =

0 commit comments

Comments
 (0)