risdenk commented on code in PR #14216:
URL: https://github.com/apache/lucene/pull/14216#discussion_r1947419192
##########
gradle/validation/error-prone.gradle:
##########
@@ -381,43 +445,58 @@ allprojects { prj ->
// '-Xep:JodaPlusMinusLong:OFF', // we don't use joda-time
// '-Xep:JodaTimeConverterManager:OFF', // we don't use joda-time
// '-Xep:JodaWithDurationAddedLong:OFF', // we don't use joda-time
+ '-Xep:JUnitIncompatibleType:WARN',
// '-Xep:LabelledBreakTarget:OFF', // stylistic
// '-Xep:LiteEnumValueOf:OFF', // we don't use protobuf
// '-Xep:LiteProtoToString:OFF', // we don't use protobuf
// '-Xep:LockNotBeforeTry:OFF', // TODO: there are problems
+ // '-Xep:LockOnNonEnclosingClassLiteral:OFF', // TODO: there are
problems
'-Xep:LogicalAssignment:WARN',
'-Xep:LongDoubleConversion:WARN',
'-Xep:LongFloatConversion:WARN',
'-Xep:LoopOverCharArray:WARN',
'-Xep:MalformedInlineTag:WARN',
// '-Xep:MathAbsoluteNegative:OFF', // TODO: there are problems
// '-Xep:MemoizeConstantVisitorStateLookups:OFF', // we don't use
this class
+ // '-Xep:MisformattedTestData:OFF', // stylistic
// '-Xep:MissingCasesInEnumSwitch:OFF', // redundant with ECJ
incompleteEnumSwitch/missingEnumCaseDespiteDefault
// '-Xep:MissingFail:OFF', // TODO: there are problems
'-Xep:MissingImplementsComparable:WARN',
// '-Xep:MissingOverride:OFF', // ECJ takes care of this
+ // '-Xep:MissingRefasterAnnotation:OFF', // we don't use this
annotation
// '-Xep:MissingSummary:OFF', // TODO: there are problems
// '-Xep:MixedMutabilityReturnType:OFF', // noisy
+ // '-Xep:MockitoDoSetup:OFF', // we don't use mocking libraries
// '-Xep:MockNotUsedInProduction:OFF', // we don't use mocking
libraries
// '-Xep:ModifiedButNotUsed:OFF', // TODO: there are problems
'-Xep:ModifyCollectionInEnhancedForLoop:WARN',
'-Xep:ModifySourceCollectionInStream:WARN',
+ '-Xep:MultimapKeys:WARN',
+ // '-Xep:MultipleNullnessAnnotations:OFF', // we don't use these
annotations
'-Xep:MultipleParallelOrSequentialCalls:WARN',
'-Xep:MultipleUnaryOperatorsInMethodCall:WARN',
+ // '-Xep:MutableGuiceModule:OFF', // we don't use guice
// '-Xep:MutablePublicArray:OFF', // TODO: there are problems
+ '-Xep:NamedLikeContextualKeyword:WARN',
'-Xep:NarrowCalculation:WARN',
// '-Xep:NarrowingCompoundAssignment:OFF', // noisy
'-Xep:NegativeCharLiteral:WARN',
'-Xep:NestedInstanceOfConditions:WARN',
// '-Xep:NewFileSystem:OFF', // we don't create new filesystems
+ // '-Xep:NonApiType:OFF', // noisy
// '-Xep:NonAtomicVolatileUpdate:OFF', // TODO: there are problems
// '-Xep:NonCanonicalType:OFF', // noisy
+ // '-Xep:NonFinalStaticField:OFF', // noisy
'-Xep:NonOverridingEquals:WARN',
+ '-Xep:NotJavadoc:WARN',
Review Comment:
This is where all the `/**` to `/*` comes from
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
