🚀 GitHub Actions update, use commit from PR, not pre-merge (#1761)

* 🔥 Remove deploy badge that won't show correctly until next release

after the fixes to the Gitter bot

* 🐛 Fix GitHub Action to upload docs artifacts with commit from PR, not pre-merge

* ♻️ Run zip docs and artifact upload only on PRs

.github/workflows/build-docs.yml

@@ -19,10 +19,12 @@ jobs:
       - name: Build Docs
         run: python3.7 ./scripts/docs.py build-all
       - name: Zip docs
+        if: github.event_name == 'pull_request'
         run: bash ./scripts/zip-docs.sh
       - uses: actions/upload-artifact@v2
+        if: github.event_name == 'pull_request'
         with:
-          name: docs-zip-${{ github.sha }}
+          name: docs-zip-${{ github.event.pull_request.head.sha }}
           path: ./docs.zip
       - name: Deploy to Netlify
         uses: nwtgck/[email protected]

README.md

@@ -8,9 +8,6 @@
 <a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3ATest" target="_blank">
     <img src="https://github.com/tiangolo/fastapi/workflows/Test/badge.svg" alt="Test">
 </a>
-<a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3APublish" target="_blank">
-    <img src="https://github.com/tiangolo/fastapi/workflows/Publish/badge.svg" alt="Publish">
-</a>
 <a href="https://codecov.io/gh/tiangolo/fastapi" target="_blank">
     <img src="https://img.shields.io/codecov/c/github/tiangolo/fastapi?color=%2334D058" alt="Coverage">
 </a>

docs/en/docs/index.md

@@ -8,9 +8,6 @@
 <a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3ATest" target="_blank">
     <img src="https://github.com/tiangolo/fastapi/workflows/Test/badge.svg" alt="Test">
 </a>
-<a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3APublish" target="_blank">
-    <img src="https://github.com/tiangolo/fastapi/workflows/Publish/badge.svg" alt="Publish">
-</a>
 <a href="https://codecov.io/gh/tiangolo/fastapi" target="_blank">
     <img src="https://img.shields.io/codecov/c/github/tiangolo/fastapi?color=%2334D058" alt="Coverage">
 </a>

docs/es/docs/index.md

@@ -8,9 +8,6 @@
 <a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3ATest" target="_blank">
     <img src="https://github.com/tiangolo/fastapi/workflows/Test/badge.svg" alt="Test">
 </a>
-<a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3APublish" target="_blank">
-    <img src="https://github.com/tiangolo/fastapi/workflows/Publish/badge.svg" alt="Publish">
-</a>
 <a href="https://codecov.io/gh/tiangolo/fastapi" target="_blank">
     <img src="https://img.shields.io/codecov/c/github/tiangolo/fastapi?color=%2334D058" alt="Coverage">
 </a>

docs/pt/docs/index.md

@@ -8,9 +8,6 @@
 <a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3ATest" target="_blank">
     <img src="https://github.com/tiangolo/fastapi/workflows/Test/badge.svg" alt="Test">
 </a>
-<a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3APublish" target="_blank">
-    <img src="https://github.com/tiangolo/fastapi/workflows/Publish/badge.svg" alt="Publish">
-</a>
 <a href="https://codecov.io/gh/tiangolo/fastapi" target="_blank">
     <img src="https://img.shields.io/codecov/c/github/tiangolo/fastapi?color=%2334D058" alt="Coverage">
 </a>

docs/zh/docs/index.md

@@ -8,9 +8,6 @@
 <a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3ATest" target="_blank">
     <img src="https://github.com/tiangolo/fastapi/workflows/Test/badge.svg" alt="Test">
 </a>
-<a href="https://github.com/tiangolo/fastapi/actions?query=workflow%3APublish" target="_blank">
-    <img src="https://github.com/tiangolo/fastapi/workflows/Publish/badge.svg" alt="Publish">
-</a>
 <a href="https://codecov.io/gh/tiangolo/fastapi" target="_blank">
     <img src="https://img.shields.io/codecov/c/github/tiangolo/fastapi?color=%2334D058" alt="Coverage">
 </a>