[gobblin] branch master updated: [GOBBLIN-1482] Move verify dependencies test to after build step to fix pegasus error (#3322)

[le...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

lesun pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/gobblin.git


The following commit(s) were added to refs/heads/master by this push:
     new 3973a84  [GOBBLIN-1482] Move verify dependencies test to after build step to fix pegasus error (#3322)
3973a84 is described below

commit 3973a8460724931aae8dd06787f42ec00d2edf3a
Author: William Lo <lo...@gmail.com>
AuthorDate: Thu Jun 24 17:37:10 2021 -0700

    [GOBBLIN-1482] Move verify dependencies test to after build step to fix pegasus error (#3322)
    
    * remove verify dependencies test
    
    * move verify dependencies to after build
    
    * remove javadoc during verification step
---
 .github/workflows/build_and_test.yaml | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/build_and_test.yaml b/.github/workflows/build_and_test.yaml
index 9da9b1e..c23efcb 100644
--- a/.github/workflows/build_and_test.yaml
+++ b/.github/workflows/build_and_test.yaml
@@ -55,12 +55,13 @@ 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
       - name: Build repository
         run: |
           ./gradlew --no-daemon clean build -x test -x javadoc -x findbugsMain -x findbugsTest -x checkstyleMain -x checkstyleJmh -x checkstyleTest -Dorg.gradle.parallel=true
+      - name: Verify Dependencies
+        run: |
+          # Since dependencies are cached, check after building if they are valid or not
+          eval ./gradlew --no-daemon assemble -x javadoc -Dorg.gradle.parallel=true
 
   static_checks:
     name: Run static checks