-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
open javac modules for google-java-format #8531
Conversation
53fe64d
to
78f57de
Compare
@@ -48,7 +48,14 @@ jobs: | |||
NEW_COMMIT: ${{ github.event.inputs.newCommit }} | |||
WORKING_DIR: /tmp/compatibility-verifier | |||
TEST_SUITE: ${{ matrix.test_suite }} | |||
MAVEN_OPTS: -Xmx2G -DskipShade -DfailIfNoTests=false -Dmaven.wagon.httpconnectionManager.ttlSeconds=25 -Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false | |||
MAVEN_OPTS: -Xmx2G -DskipShade -DfailIfNoTests=false -Dmaven.wagon.httpconnectionManager.ttlSeconds=25 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this multi-line with backslash the correct format?
or you can try this one:
MAVEN_OPTS: >
-Xmx2G -DskipShade -DfailIfNoTests=false -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false
...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may also need to modify the script a bit to let it trigger the job once to kick off the action tasks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch. how do I force it to run?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just add a newline in pom.xml should trigger it. You can revert it back once the test is done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
running now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
running now
78f57de
to
1facf14
Compare
1facf14
to
1d41049
Compare
@xiangfu0 it builds. I will revert the dummy commit. |
👍 |
1d41049
to
c24c741
Compare
Google-java-format is JK17 incompatible, having dependencies on Javac internals, so needs some modules exported to run on JDK17. See more here: google/google-java-format#612