Add Spotless for Code Formatting (#2613)
* style: format from cli * ci: add style check
This commit is contained in:
committed by
GitHub
parent
21974fce0e
commit
a6d94919e9
@@ -61,5 +61,12 @@ jobs:
|
|||||||
- name: Check file license headers
|
- name: Check file license headers
|
||||||
run: mvn license:check -PcheckLicense --batch-mode
|
run: mvn license:check -PcheckLicense --batch-mode
|
||||||
|
|
||||||
|
- name: Check code style
|
||||||
|
run: |
|
||||||
|
# compare style to target branch
|
||||||
|
git remote add target ${{ github.event.pull_request.base.repo.clone_url }}
|
||||||
|
git fetch target
|
||||||
|
mvn spotless:check -DratchetFrom=${{ github.event.pull_request.base.sha }}
|
||||||
|
|
||||||
- name: Run tests & javadoc
|
- name: Run tests & javadoc
|
||||||
run: mvn clean verify javadoc:javadoc -DdetectOfflineLinks=false -PgenerateTestReport ${{ inputs.maven_properties }} --batch-mode
|
run: mvn clean verify javadoc:javadoc -DdetectOfflineLinks=false -PgenerateTestReport ${{ inputs.maven_properties }} --batch-mode
|
||||||
20
pom.xml
20
pom.xml
@@ -130,6 +130,7 @@
|
|||||||
<!-- Dash tool integration - END -->
|
<!-- Dash tool integration - END -->
|
||||||
|
|
||||||
<skipPublishing>false</skipPublishing>
|
<skipPublishing>false</skipPublishing>
|
||||||
|
<ratchetFrom>origin/master</ratchetFrom>
|
||||||
</properties>
|
</properties>
|
||||||
|
|
||||||
<developers>
|
<developers>
|
||||||
@@ -368,6 +369,25 @@
|
|||||||
<groupId>org.jacoco</groupId>
|
<groupId>org.jacoco</groupId>
|
||||||
<artifactId>jacoco-maven-plugin</artifactId>
|
<artifactId>jacoco-maven-plugin</artifactId>
|
||||||
</plugin>
|
</plugin>
|
||||||
|
<plugin>
|
||||||
|
<!--
|
||||||
|
https://github.com/diffplug/spotless/tree/main/plugin-maven#-spotless-keep-your-code-spotless-with-maven
|
||||||
|
|
||||||
|
Usage:
|
||||||
|
mvn spotless:apply
|
||||||
|
mvn spotless:check
|
||||||
|
-->
|
||||||
|
<groupId>com.diffplug.spotless</groupId>
|
||||||
|
<artifactId>spotless-maven-plugin</artifactId>
|
||||||
|
<configuration>
|
||||||
|
<ratchetFrom>${ratchetFrom}</ratchetFrom>
|
||||||
|
<java>
|
||||||
|
<eclipse>
|
||||||
|
<file>${maven.multiModuleProjectDirectory}/eclipse_codeformatter.xml</file>
|
||||||
|
</eclipse>
|
||||||
|
</java>
|
||||||
|
</configuration>
|
||||||
|
</plugin>
|
||||||
</plugins>
|
</plugins>
|
||||||
|
|
||||||
<pluginManagement>
|
<pluginManagement>
|
||||||
|
|||||||
Reference in New Issue
Block a user