Преглед изворни кода

Merge pull request #51 from nameczz/main

fix(pencil): Fix ci
ryjiang пре 4 година
родитељ
комит
aefbdbd437
1 измењених фајлова са 10 додато и 4 уклоњено
  1. 10 4
      .github/workflows/release.yml

+ 10 - 4
.github/workflows/release.yml

@@ -1,8 +1,11 @@
 on:
-  pull_request:
+  push:
     branches:
       - main
-    types: [closed]
+  # pull_request:
+  #   branches:
+  #     - main
+  #   types: [closed]
 
 jobs:
   publish:
@@ -14,8 +17,11 @@ jobs:
         with:
           node-version: 12
 
-      - name: Docker login
-        run: echo '${{ secrets.DOCKER_PWD }}' | docker login --username ${{ secrets.DOCKER_USERNAME }} --password-stdin
+      - name: Login to DockerHub
+        uses: docker/login-action@v1
+        with:
+          username: ${{ secrets.DOCKER_USERNAME }}
+          password: ${{ secrets.DOCKER_PWD }}
 
       - name: Docker Build
         run: docker build -t milvusdb/milvus-insight:${GITHUB_REF#refs/tags/} --build-arg VERSION=${GITHUB_REF#refs/tags/} .