Merge pull request #21 from reactioncommerce/docker-publish-update

Add condition to not publish for forks
reactioncommerce · Mar 8, 2018 · 8abd2a5 · 8abd2a5
2 parents 8f95ed1 + 0a820b5
commit 8abd2a5
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -87,11 +87,15 @@ jobs:
       - run:
           name: Docker Push
           command: |
-            docker login -u "$DOCKER_USER" -p "$DOCKER_PASS"
-            docker push "$DOCKER_REPOSITORY:$CIRCLE_SHA1"
-            cat docker-cache/docker-tags.txt \
-              | xargs -t -I % \
-                docker push "$DOCKER_REPOSITORY:%"
+            if [ -z "$CIRCLE_PR_USERNAME" ]; then \
+              docker login -u "$DOCKER_USER" -p "$DOCKER_PASS"
+              docker push "$DOCKER_REPOSITORY:$CIRCLE_SHA1"
+              cat docker-cache/docker-tags.txt \
+                | xargs -t -I % \
+                  docker push "$DOCKER_REPOSITORY:%"
+            else
+              echo "No deploy for forks"
+            fi
 
   test:
     <<: *defaults