From 30d212da65d785abd0004a0c3f5ddae79d123afd Mon Sep 17 00:00:00 2001
From: Pascal Hartig <phartig@rdrei.net>
Date: Mon, 16 Nov 2020 11:04:03 -0800
Subject: [PATCH] Select release commit from larger push (#1678)

Summary:
GitHub only issues one push event for multiple commits.
This causes things to behave weirdly. My action now
looks for a commit with a certain message and runs
the subsequent steps on it. If there's more than one
matching commit, it uses the last one. Not great, not terrible.

Pull Request resolved: https://github.com/facebook/flipper/pull/1678

Test Plan: Ran it on my fork: https://github.com/passy/flipper-1/actions/runs/360332680

Reviewed By: nikoant

Differential Revision: D24955986

Pulled By: passy

fbshipit-source-id: 6dcbe11e69bf77c803e3907ced3185ca85483ec8
---
 .github/workflows/release.yml | 26 ++++++++++++++++++++++++--
 1 file changed, 24 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
index 65216ed76..56191aa5b 100644
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -12,12 +12,21 @@ jobs:
     outputs:
       tag: ${{ steps.tag-version-commit.outputs.tag }}
     steps:
+      - uses: passy/extract-version-commit@main
+        id: extract-version-commit
+        with:
+          version_regex: '^Flipper Release: v([0-9]+\.[0-9]+\.[0-9]+)(?:\n|$)'
       - uses: actions/checkout@v2
+        if: ${{ steps.extract-version-commit.outputs.commit != ''}}
+        with:
+          ref: ${{ steps.extract-version-commit.outputs.commit }}
       - name: Tag version commit
+        if: ${{ steps.extract-version-commit.outputs.commit != ''}}
         id: tag-version-commit
-        uses: passy/tag-version-commit@v1.1.1
+        uses: passy/tag-version-commit@master
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
+          ref: ${{ steps.extract-version-commit.outputs.commit }}
           version_tag_prefix: 'v'
           version_assertion_command: 'grep -q "\"version\": \"$version\"" desktop/package.json'
       - name: Create release
@@ -35,12 +44,16 @@ jobs:
           prerelease: false
 
   build-mac:
+    needs:
+      - release
     runs-on: macos-latest
     env:
       desktop-directory: ./desktop
 
     steps:
     - uses: actions/checkout@v2
+      with:
+        ref: ${{ needs.release.outputs.tag }}
     - uses: actions/setup-node@v1
       with:
         node-version: '12.x'
@@ -63,12 +76,16 @@ jobs:
         path: 'dist/Flipper-mac.dmg'
 
   build-linux:
+    needs:
+      - release
     runs-on: ubuntu-latest
     env:
       desktop-directory: ./desktop
 
     steps:
     - uses: actions/checkout@v2
+      with:
+        ref: ${{ needs.release.outputs.tag }}
     - uses: actions/setup-node@v1
       with:
         node-version: '12.x'
@@ -91,12 +108,16 @@ jobs:
         path: 'dist/Flipper-linux.zip'
 
   build-win:
+    needs:
+      - release
     runs-on: windows-latest
     env:
       desktop-directory: ./desktop
 
     steps:
     - uses: actions/checkout@v2
+      with:
+        ref: ${{ needs.release.outputs.tag }}
     - uses: actions/setup-node@v1
       with:
         node-version: '12.x'
@@ -119,12 +140,12 @@ jobs:
         path: 'dist/Flipper-win.zip'
 
   publish:
-    runs-on: ubuntu-latest
     needs:
       - build-win
       - build-linux
       - build-mac
       - release
+    runs-on: ubuntu-latest
 
     steps:
     - name: Download Mac
@@ -166,3 +187,4 @@ jobs:
       with:
         workflow: Publish Pods
         token: ${{ secrets.PERSONAL_TOKEN }}
+        ref: ${{ needs.release.outputs.tag }}