lanyuanxiaoyao/hudi
1
0
Fork 0
Code Activity

[MINOR] Fixed checkstyle config to be based off Maven root-dir (requires Maven >=3.3.1 to work properly); (#4009)

Browse Source 
Updated README
This commit is contained in:
Alexey Kudinkin
2021-11-16 18:30:16 -08:00
committed by GitHub
parent 6f5e661010
commit cbcbec4d38
3 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
README.md
View File

@@ -51,7 +51,7 @@ Prerequisites for building Apache Hudi:
* Unix-like system (like Linux, Mac OS X) * Unix-like system (like Linux, Mac OS X)
* Java 8 (Java 9 or 10 may work) * Java 8 (Java 9 or 10 may work)
* Git * Git
* Maven * Maven (>=3.3.1)
``` ```
# Checkout code and build # Checkout code and build

2
pom.xml
View File

@@ -231,6 +231,8 @@
<sourceDirectories> <sourceDirectories>
<sourceDirectory>${project.build.sourceDirectory}</sourceDirectory> <sourceDirectory>${project.build.sourceDirectory}</sourceDirectory>
</sourceDirectories> </sourceDirectories>
<!-- NOTE: This property is only available in Maven >= 3.3.1 -->
<propertyExpansion>basedir=${maven.multiModuleProjectDirectory}</propertyExpansion>
<excludes>**\/generated-sources\/</excludes> <excludes>**\/generated-sources\/</excludes>
</configuration> </configuration>
<executions> <executions>

2
style/checkstyle.xml
View File

@@ -305,7 +305,7 @@
<module name="SimplifyBooleanExpression"/> <module name="SimplifyBooleanExpression"/>
<module name="ImportControl"> <module name="ImportControl">
<property name="file" value="style/import-control.xml"/> <property name="file" value="${basedir}/style/import-control.xml"/>
<property name="path" value="^.*[\\/]hudi-client[\\/]hudi-client-common[\\/]src[\\/].*$"/> <property name="path" value="^.*[\\/]hudi-client[\\/]hudi-client-common[\\/]src[\\/].*$"/>
</module> </module>