1- <?xml version =" 1.0"
2- <!DOCTYPE module PUBLIC
3- "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN"
4- "https://checkstyle.org/dtds/configuration_1_3.dtd">
5-
6- <!--
1+ <?xml version =" 1.0" encoding =" UTF-8" standalone =" no" <!--
72 Palantir Baseline Checkstyle configuration.
83 Authors: Robert Fink, Brian Worth, Merrick Zoubeiri, and many other contributors. Based in part on http://checkstyle.sourceforge.net/google_style.html
94 Please keep checks alphabetized with one exception: "relaxed" checks are grouped together at the bottom for easier disabling.
105 Check-specific comments reference documents internal to Palantir and can be safely ignored or removed.
11- -->
12-
6+ --> <!DOCTYPE module PUBLIC "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN" "https://checkstyle.org/dtds/configuration_1_3.dtd">
137<module name =" Checker"
148 <property name =" charset" value =" UTF-8"
159 <property name =" severity" value =" error"
1812 <module name =" NewlineAtEndOfFile" <!-- Java Style Guide: Line ending: LF -->
1913 <property name =" lineSeparator" value =" lf"
2014 </module >
21- <module name =" RegexpHeader"
22- <property name =" header" value =" ^/\*$\n^ \* \(c\) Copyright \d{4} Palantir Technologies Inc\. All rights reserved\.$"
23- <property name =" fileExtensions" value =" .java,.ts"
24- </module >
25- <module name =" RegexpMultiline" <!-- Development Practices: Writing good unit tests -->
26- <property name =" fileExtensions" value =" java"
27- <property name =" format" value =" @VisibleForTesting\s+(protected|public)"
28- <property name =" message" value =" @VisibleForTesting members should be package-private."
29- </module >
3015 <module name =" RegexpSingleline" <!-- No reference needed as this is evident. -->
3116 <property name =" format" value =" <<<<<<< "
3217 <property name =" message" value =" Found (<<<<<<< ), so it looks like you had a merge conflict that compiles. Please fix it."
5641 <module name =" BeforeExecutionExclusionFileFilter"
5742 <property name =" fileNamePattern" value =" module\-info\.java$"
5843 </module >
44+ <module name =" LineLength" <!-- Java Style Guide: No line-wrapping -->
45+ <property name =" max" value =" 120"
46+ <property name =" ignorePattern" value =" ^package.*|^import.*|a href|href|http://|https://|ftp://"
47+ </module >
5948 <module name =" TreeWalker"
6049 <module name =" SuppressionCommentFilter" <!-- baseline-gradle: README.md -->
6150 <module name =" SuppressionCommentFilter"
8675 </module >
8776 <module name =" AvoidStarImport" <!-- Java Style Guide: No wildcard imports -->
8877 <module name =" AvoidStaticImport" <!-- Java Style Guide: No static imports -->
89- <property name =" excludes" value ="
90- com.google.common.base.Preconditions.*,
91- com.palantir.logsafe.Preconditions.*,
92- java.util.Collections.*,
93- java.util.stream.Collectors.*,
94- org.apache.commons.lang3.Validate.*,
95- org.assertj.core.api.Assertions.*,
96- org.mockito.Mockito.*" />
78+ <property name =" excludes" value =" com.google.common.base.Preconditions.*, com.palantir.logsafe.Preconditions.*, java.util.Collections.*, java.util.stream.Collectors.*, org.apache.commons.lang3.Validate.*, org.assertj.core.api.Assertions.*, org.mockito.Mockito.*"
9779 </module >
9880 <module name =" ClassTypeParameterName" <!-- Java Style Guide: Type variable names -->
9981 <property name =" format" value =" (^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"
143125 <module name =" IllegalImport" <!-- Java Coding Guidelines: Import the canonical package -->
144126 <property name =" id" value =" BanShadedClasses"
145127 <property name =" illegalPkgs" value =" .*\.(repackaged|shaded|thirdparty)"
146- <property name =" regexp" value =" true" />
128+ <property name =" regexp" value =" true"
147129 <message key =" import.illegal" value =" Must not import repackaged classes."
148130 </module >
149131 <module name =" IllegalImport"
150132 <property name =" illegalPkgs" value =" ^org\.gradle\.(internal|.*\.internal)"
151- <property name =" regexp" value =" true" />
133+ <property name =" regexp" value =" true"
152134 <message key =" import.illegal" value =" Do not rely on gradle internal classes as these may change in minor releases - use org.gradle.api versions instead."
153135 </module >
154136 <module name =" IllegalImport"
201183 <property name =" separated" value =" true"
202184 <property name =" sortStaticImportsAlphabetically" value =" true"
203185 </module >
186+
204187 <module name =" InnerAssignment" <!-- Java Coding Guidelines: Inner assignments: Not used -->
205- <module name =" LeftCurly" <!-- Java Style Guide: Nonempty blocks: K & R style -->
206- <module name =" LineLength" <!-- Java Style Guide: No line-wrapping -->
207- <property name =" max" value =" 120"
208- <property name =" ignorePattern" value =" ^package.*|^import.*|a href|href|http://|https://|ftp://"
209- </module >
188+ <!-- Java Style Guide: Nonempty blocks: K & R style -->
210189 <module name =" MemberName" <!-- Java Style Guide: Non-constant field names -->
211190 <property name =" format" value =" ^[a-z][a-zA-Z0-9]+$"
212191 <message key =" name.invalidPattern" value =" Member name ''{0}'' must match pattern ''{1}''."
247226 <message key =" name.invalidPattern" value =" Package name ''{0}'' must match pattern ''{1}''."
248227 </module >
249228 <module name =" ParameterAssignment" <!-- Java Coding Guidelines: Final variables and parameters -->
250- < module name = " ParenPad " /> <!-- Java Style Guide: Horizontal whitespace -->
229+ <!-- Java Style Guide: Horizontal whitespace -->
251230 <module name =" RedundantImport" <!-- Java Style Guide: No unused imports -->
252231 <module name =" RedundantModifier" <!-- Java Coding Guidelines: Avoid redundant modifiers -->
253232 <module name =" RegexpSinglelineJava" <!-- Java Coding Guidelines: Use appropriate assertion methods -->
363342 </module >
364343 <module name =" RegexpSinglelineJava"
365344 <property name =" format" value =" \bCharset.defaultCharset\("
366- <property name =" message" value =" Use explicit charset (e.g. StandardCharsets.UTF-8 ) instead of default."
345+ <property name =" message" value =" Use explicit charset (e.g. StandardCharsets.UTF_8 ) instead of default."
367346 </module >
368347 <module name =" RegexpSinglelineJava"
369348 <property name =" format" value =" \bIOUtils\.toString\("
372351 <module name =" RegexpSinglelineJava"
373352 <property name =" format" value =" \/\/TODO|\/\/ TODO(?!\([^()\s]+\): )"
374353 <property name =" message" value =" TODO format: // TODO(#issue): explanation"
354+ <property name =" ignoreCase" value =" true"
375355 </module >
376356 <module name =" RegexpSinglelineJava"
377357 <property name =" format" value =" (void setUp\(\))|(void setup\(\))|(void setupStatic\(\))|(void setUpStatic\(\))|(void beforeTest\(\))|(void teardown\(\))|(void tearDown\(\))|(void beforeStatic\(\))|(void afterStatic\(\))"
378358 <property name =" message" value =" Test setup/teardown methods are called before(), beforeClass(), after(), afterClass(), but not setUp, teardown, etc."
379359 </module >
380360 <module name =" RightCurly" <!-- Java Style Guide: Nonempty blocks: K & R style -->
381361 <property name =" option" value =" same"
382- <property name =" tokens" value =" LITERAL_TRY, LITERAL_CATCH, LITERAL_FINALLY, LITERAL_IF, LITERAL_ELSE, LITERAL_DO"
362+ <property name =" tokens" value =" CLASS_DEF, METHOD_DEF, CTOR_DEF, LITERAL_TRY, LITERAL_CATCH, LITERAL_FINALLY, LITERAL_IF, LITERAL_ELSE, LITERAL_DO"
383363 </module >
384364 <module name =" RightCurly" <!-- Java Style Guide: Nonempty blocks: K & R style -->
385365 <property name =" option" value =" alone"
386- <property name =" tokens" value =" CLASS_DEF, METHOD_DEF, CTOR_DEF, LITERAL_FOR, LITERAL_WHILE, STATIC_INIT, INSTANCE_INIT"
366+ <property name =" tokens" value =" LITERAL_FOR, LITERAL_WHILE, STATIC_INIT, INSTANCE_INIT"
387367 </module >
388368 <module name =" SeparatorWrap" <!-- Java Style Guide: Where to break -->
389369 <property name =" tokens" value =" DOT"
401381 <module name =" SuppressWarnings"
402382 <property name =" format" value =" serial"
403383 </module >
404- <module name =" SuppressWarningsHolder" /> <!-- Required for SuppressWarningsFilter -->
384+ <module name =" SuppressWarningsHolder" <!-- Required for SuppressWarningsFilter -->
405385 <module name =" TypeName" <!-- Java Style Guide: Class names -->
406386 <message key =" name.invalidPattern" value =" Type name ''{0}'' must match pattern ''{1}''."
407387 </module >
408388 <module name =" TypecastParenPad" <!-- Java Style Guide: Horizontal whitespace -->
409- <module name =" UnnecessaryParentheses"
410389 <module name =" UnusedImports" <!-- Java Style Guide: No unused imports -->
411390 <property name =" processJavadoc" value =" true"
412391 </module >
413392 <module name =" UpperEll" <!-- Java Style Guide: Numeric Literals -->
414393 <module name =" VisibilityModifier" <!-- Java Coding Guidelines: Minimize mutability -->
415394 <module name =" WhitespaceAfter" <!-- Java Style Guide: Horizontal whitespace -->
416- <module name =" WhitespaceAround" <!-- Java Style Guide: Horizontal whitespace -->
417- <property name =" allowEmptyConstructors" value =" true"
418- <property name =" allowEmptyMethods" value =" true"
419- <property name =" allowEmptyTypes" value =" true"
420- <property name =" allowEmptyLoops" value =" true"
421- <property name =" allowEmptyLambdas" value =" true"
422- <property name =" ignoreEnhancedForColon" value =" false"
423- <message key =" ws.notFollowed" value =" WhitespaceAround: ''{0}'' is not followed by whitespace. Empty blocks may only be represented as '{}' when not part of a multi-block statement (4.1.3)"
424- <message key =" ws.notPreceded" value =" WhitespaceAround: ''{0}'' is not preceded with whitespace."
425- </module >
395+
426396
427397 <!-- Stricter checks begin: delete some or all of the following for faster prototyping, but please restore before pushing to production. -->
428398
437407 <module name =" JavadocMethod" <!-- Java Style Guide: Where Javadoc is used -->
438408 <property name =" scope" value =" public"
439409 <property name =" allowMissingParamTags" value =" true"
440- <property name =" allowMissingThrowsTags" value =" true"
441410 <property name =" allowMissingReturnTag" value =" true"
442- <property name =" minLineCount" value =" 99999999"
443411 <property name =" allowedAnnotations" value =" Override, Test"
444- <property name =" allowThrowsTagsForSubclasses" value =" true"
445412 </module >
446413 <module name =" JavadocStyle" <!-- Java Style Guide: Javadoc -->
447414 <module name =" JavadocTagContinuationIndentation" <!-- Java Style Guide: At-clauses -->
474441
475442 <!-- Stricter checks end -->
476443 </module >
477- </module >
444+ </module >
0 commit comments