aplex commented on a change in pull request #3307:
URL: https://github.com/apache/gobblin/pull/3307#discussion_r652006580
##########
File path: .github/workflows/build_and_test.yaml
##########
@@ -49,6 +49,9 @@ jobs:
# Only rebuild cache if build.gradle is changed
key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }}
restore-keys: ${{ runner.os }}-gradle
+ - name: Verify Dependencies
+ run: |
+ eval ./gradlew assemble
Review comment:
I see that we skip test on the normal build below. I think that will
also skip compilation and dependency resolution for test code. We can tell
gradle to compile tests but not run them yet like this -
https://stackoverflow.com/questions/43638697/gradle-compile-tests-but-do-not-run-them
Did we see those dependency problems in test or in a main code?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
