commit 89549e7f8d13be719c26b90e804810541fa42662
author Steve Lawrence <slawrence@apache.org> Mon May 10 07:18:25 2021 -0400
committer Steve Lawrence <stephen.d.lawrence@gmail.com> Mon May 10 08:48:25 2021 -0400
tree 5144065334a4349638a84588afa97b7e3e8964b4
parent 1c824761a6df3c59b5c61d0bef5b4b7375141917

Only publish site updates from the master branch

We currently enable the publish action when commits are made directly to
the apache/daffodil-site repo. This prevents publishing pull requests
until they are merged. However, if we push a branch directly to
apache/daffodil-site for a pull request rather than a fork, then this
action is still triggered, and we will publish from the branch. We don't
want that. We only want to ever publish commits made to the master
branch of the apache/daffodil repo. This fixes that.

.github/workflows/main.yml

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
index 8d9544f..b8d9e24 100644
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -54,7 +54,7 @@
           jekyll build --source site
 
       - name: Publish
-        if: github.repository == 'apache/daffodil-site'
+        if: ${{ github.repository == 'apache/daffodil-site' && github.ref == 'refs/heads/master' }}
         run: |
           git checkout asf-site
           rm -rf content