Browse Source

Merge pull request #61 from nameczz/main

fix ci action
nameczz 4 years ago
parent
commit
94dc0eba19
1 changed files with 2 additions and 1 deletions
  1. 2 1
      .github/workflows/dev.yml

+ 2 - 1
.github/workflows/dev.yml

@@ -9,7 +9,6 @@ on:
 jobs:
   publish:
     runs-on: ubuntu-latest
-    if: github.event.pull_request.merged == true
     steps:
       - uses: actions/checkout@v2
       - name: Setup Node.js
@@ -24,7 +23,9 @@ jobs:
           password: ${{ secrets.DOCKER_PWD }}
 
       - name: Docker Build
+        if: github.event.pull_request.merged == true
         run: docker build -t milvusdb/milvus-insight:dev --build-arg VERSION=dev .
 
       - name: Docker Push Dev
+        if: github.event.pull_request.merged == true
         run: docker push milvusdb/milvus-insight:dev