From 70f2a5e70dcec3a44176c7ef9b6be68456f392ad Mon Sep 17 00:00:00 2001
From: Alex Collins <alex_collins@intuit.com>
Date: Thu, 31 Oct 2019 11:00:56 -0700
Subject: [PATCH] changes on circleci to .circleci/config.yml,

---
 .circleci/config.yml | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index 3177f9ef..1eedc93b 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -6,20 +6,25 @@ jobs:
     steps:
       - checkout
       - run: ct lint --config .circleci/chart-testing.yaml
+  # Technically this only needs to be run on master, but it's good to have it run on every PR
+  # so that it is regularly tested.
   publish:
     docker:
+      # We just need an image with `helm` on it. Handily we know of one already.
       - image: gcr.io/kubernetes-charts-ci/test-image:v3.0.1
     steps:
+      # install the additional keys needed to push to Github. Alex Collins owns these keys.
       - add_ssh_keys
       - checkout
       - run: helm init --client-only
       - run: git config --global user.email "nobody@circleci.com"
       - run: git config --global user.name "Circle CI Build"
-      - run: sh ./scripts/publish.sh
+      # Only actually publish charts on master.
       - run: |
           if [ "$CIRCLE_BRANCH" = "circleci" ]; then
-           git push origin gh-pages
+           echo 'export GIT_PUSH=true' >> $BASH_ENV 
           fi
+      - run: sh ./scripts/publish.sh
 workflows:
   version: 2
   workflow: