Skip to content
This repository was archived by the owner on Jul 6, 2023. It is now read-only.

Commit a908498

Browse files
loveleifloveleif
authored
Merge pull request #265 from loveleif/4.0-code-style
Introduce codestyle
2 parents e332f55 + 1876af7 commit a908498

File tree

123 files changed

+9390
-5334
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

123 files changed

+9390
-5334
lines changed

‎build.gradle‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ buildscript {
1111

1212
allprojects {
1313
apply plugin: 'java'
14+
apply plugin: 'checkstyle'
1415

1516
sourceCompatibility = 1.8
1617
targetCompatibility = 1.8

‎config/checkstyle/checkstyle.xml‎

Lines changed: 111 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,111 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
<!DOCTYPE module PUBLIC
3+
"-//Puppy Crawl//DTD Check Configuration 1.2//EN"
4+
"http://www.puppycrawl.com/dtds/configuration_1_2.dtd">
5+
6+
<module name="Checker">
7+
8+
<property name="charset" value="UTF-8"/>
9+
<property name="fileExtensions" value="java"/>
10+
11+
<!-- Require license headers (GPL, AGPL or Apache 2.0) -->
12+
<module name="RegexpSingleline">
13+
<property name="format"
14+
value="^(\s|\*)*Copyright \(c\) 2002-[0-9]{4} &quot;Neo4j,&quot;"/>
15+
<property name="minimum" value="1"/>
16+
<property name="maximum" value="1"/>
17+
<property name="message" value="Missing, wrong or duplicated license header"/>
18+
</module>
19+
20+
<!-- Prohibit tabs -->
21+
<module name="FileTabCharacter">
22+
<property name="eachLine" value="true"/>
23+
</module>
24+
25+
<!-- Require new line at the end of file -->
26+
<module name="NewlineAtEndOfFile">
27+
<property name="lineSeparator" value="lf_cr_crlf" />
28+
</module>
29+
30+
<!-- Prohibit trailing spaces -->
31+
<module name="RegexpSingleline">
32+
<property name="format" value="\s+$"/>
33+
<property name="minimum" value="0"/>
34+
<property name="maximum" value="0"/>
35+
<property name="message" value="Line has trailing spaces."/>
36+
</module>
37+
38+
<!-- Prohibit consecutive empty lines (except the lines after package/import) -->
39+
<module name="RegexpMultiline">
40+
<property name="format" value="\n *(?!package )(?!import )[^\n]+\n{3,}"/>
41+
<property name="message" value="Two or more consecutive empty lines"/>
42+
</module>
43+
44+
<module name="LineLength">
45+
<property name="fileExtensions" value="java" />
46+
<property name="max" value="160"/>
47+
<property name="ignorePattern" value="a href|href|http://|https://"/>
48+
</module>
49+
50+
<module name="TreeWalker">
51+
52+
<module name="PackageDeclaration"/>
53+
<module name="UpperEll"/>
54+
<module name="ArrayTypeStyle"/>
55+
<module name="MissingOverride"/>
56+
<module name="EmptyStatement"/>
57+
<module name="SuperFinalize"/>
58+
<module name="EqualsHashCode"/>
59+
<module name="ModifierOrder"/>
60+
<module name="RedundantImport"/>
61+
<module name="MissingSwitchDefault"/>
62+
<module name="DefaultComesLast"/>
63+
<module name="MethodParamPad"/>
64+
<module name="TypecastParenPad"/>
65+
<module name="EmptyCatchBlock">
66+
<property name="exceptionVariableName" value="ignore|ignored"/>
67+
</module>
68+
<module name="AnnotationLocation">
69+
<property name="allowSamelineMultipleAnnotations" value="false"/>
70+
<property name="allowSamelineSingleParameterlessAnnotation" value="false"/>
71+
<property name="allowSamelineParameterizedAnnotation" value="false"/>
72+
</module>
73+
<module name="UnnecessaryParentheses"/>
74+
<module name="LeftCurly">
75+
<property name="option" value="nl"/>
76+
<property name="tokens" value="INTERFACE_DEF, CLASS_DEF, ANNOTATION_DEF, ENUM_DEF, CTOR_DEF, METHOD_DEF, ENUM_CONSTANT_DEF, LITERAL_WHILE, LITERAL_TRY, LITERAL_CATCH, LITERAL_FINALLY, LITERAL_SYNCHRONIZED, LITERAL_SWITCH, LITERAL_DO, LITERAL_IF, LITERAL_ELSE, LITERAL_FOR, STATIC_INIT, OBJBLOCK"/>
77+
</module>
78+
<module name="RightCurly">
79+
<property name="option" value="alone"/>
80+
</module>
81+
<module name="NeedBraces"/>
82+
<module name="RedundantModifier"/>
83+
<module name="WhitespaceAround">
84+
<property name="allowEmptyConstructors" value="true"/>
85+
<property name="allowEmptyLambdas" value="true"/>
86+
<property name="allowEmptyMethods" value="true"/>
87+
</module>
88+
<module name="OneStatementPerLine">
89+
<property name="treatTryResourcesAsStatement" value="true"/>
90+
</module>
91+
<module name="ParenPad">
92+
<property name="tokens"
93+
value="ANNOTATION, ANNOTATION_FIELD_DEF, CTOR_CALL, CTOR_DEF, ENUM_CONSTANT_DEF, LITERAL_CATCH, LITERAL_DO, LITERAL_FOR, LITERAL_IF, LITERAL_NEW, LITERAL_SWITCH, LITERAL_SYNCHRONIZED, LITERAL_WHILE, RESOURCE_SPECIFICATION, SUPER_CTOR_CALL, LAMBDA, METHOD_DEF"/>
94+
<property name="option" value="space"/>
95+
</module>
96+
<module name="AvoidStarImport"/>
97+
<module name="GenericWhitespace"/>
98+
<module name="ExplicitInitialization"/>
99+
100+
<!-- Included in mono repo but not worth it to introduce here at this moment
101+
<module name="IllegalImport">
102+
<property name="illegalPkgs" value="junit.framework"/>
103+
</module>
104+
-->
105+
106+
<module name="LocalVariableName">
107+
<property name="allowOneCharVarInForLoop" value="true"/>
108+
</module>
109+
</module>
110+
111+
</module>

‎cypher-shell/build.gradle‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,3 +70,7 @@ dependencies {
7070
testCompile "org.hamcrest:java-hamcrest:$hamcrestVersion"
7171
testCompileOnly "com.google.code.findbugs:annotations:$findbugsVersion"
7272
}
73+
74+
checkstyle {
75+
toolVersion = '8.35'
76+
}

0 commit comments

Comments
 (0)