Skip to content

Commit

Permalink
Enable error prone checks in GitHub workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
electrum committed Nov 23, 2019
1 parent 6880027 commit 7949e41
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 1 deletion.
17 changes: 17 additions & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,23 @@ jobs:
- name: Test Docker Image
run: docker/build-local.sh

error-prone-checks:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
- uses: actions/setup-java@v1
with:
java-version: 8
- name: Maven Install
run: |
export MAVEN_OPTS="${MAVEN_INSTALL_OPTS}"
./mvnw install ${MAVEN_FAST_INSTALL} -pl '!presto-docs,!presto-server,!presto-server-rpm'
- name: Error Prone Checks
run: |
export MAVEN_OPTS="${MAVEN_INSTALL_OPTS}"
./mvnw -B -e -T C1 clean test-compile -Dair.check.skip-all -P errorprone-compiler-presto \
-pl '!presto-docs,!presto-server,!presto-server-rpm'
web-ui-checks:
runs-on: ubuntu-latest
steps:
Expand Down
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ script:
fi
- |
if [[ -v MAVEN_CHECKS ]]; then
env MAVEN_OPTS="${MAVEN_INSTALL_OPTS}" ./mvnw clean compile test-compile -DskipTests -B -T C1 -P errorprone-compiler-presto \
env MAVEN_OPTS="${MAVEN_INSTALL_OPTS}" ./mvnw clean test-compile -DskipTests -Dair.check.skip-all -B -T C1 -P errorprone-compiler-presto \
-pl '!presto-server-rpm,!presto-server,!presto-docs'
fi
- |
Expand Down
1 change: 1 addition & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -1454,6 +1454,7 @@
<version>3.8.0</version>
<!--suppress MavenModelInspection -->
<configuration combine.children="merge">
<fork>true</fork>
<compilerArgs>
<arg>-J-Xbootclasspath/p:${settings.localRepository}/com/google/errorprone/javac/${javac.version}/javac-${javac.version}.jar</arg>
<arg>-XDcompilePolicy=simple</arg>
Expand Down

0 comments on commit 7949e41

Please sign in to comment.