ci: extract parts from workflows into actions

Signed-off-by: Jérémie Drouet <jeremie.drouet@gmail.com>
jdrouet · Jun 27, 2024 · de5a530 · de5a530
1 parent 496641d
commit de5a530
Show file tree

Hide file tree

Showing 4 changed files with 59 additions and 56 deletions.
diff --git a/.github/actions/download-binaries/action.yml b/.github/actions/download-binaries/action.yml
@@ -0,0 +1,14 @@
+name: download-binaries
+description: download the binaries that were previously built
+runs:
+  using: "composite"
+  steps:
+    - uses: actions/download-artifact@v4
+      with:
+        name: binary-windows
+    - uses: actions/download-artifact@v4
+      with:
+        name: binary-darwin
+    - uses: actions/download-artifact@v4
+      with:
+        name: binary-linux
diff --git a/.github/actions/persist-metrics/action.yml b/.github/actions/persist-metrics/action.yml
@@ -0,0 +1,32 @@
+name: persist-metrics
+description: save binary sizes with git-metrics
+inputs:
+  push:
+    description: "Should we push metrics"
+    required: true
+    default: "false"
+runs:
+  using: "composite"
+  steps:
+    - name: set git user
+      uses: ./.github/actions/git-user
+    - name: make git-metrics executable
+      shell: bash
+      run: chmod +x ./git-metrics_linux-x86_64
+
+    - name: pull metrics
+      shell: bash
+      run: ./git-metrics_linux-x86_64 --backend command pull
+    - name: set metrics
+      shell: bash
+      run: |
+        ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-pc-windows-msvc" --tag "platform.os: windows" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_win-x86_64.exe)
+        ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-pc-windows-msvc" --tag "platform.os: windows" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_win-aarch64.exe)
+        ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-apple-darwin" --tag "platform.os: macos" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_darwin-x86_64)
+        ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-apple-darwin" --tag "platform.os: macos" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_darwin-aarch64)
+        ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-unknown-linux-gnu" --tag "platform.os: linux" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_linux-x86_64)
+        ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-unknown-linux-gnu" --tag "platform.os: linux" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_linux-aarch64)
+    - name: push metrics
+      if: ${{ inputs.push == 'true' }}
+      shell: bash
+      run: ./git-metrics_linux-x86_64 --backend command push
diff --git a/.github/workflows/pull-request-build.yml b/.github/workflows/pull-request-build.yml
@@ -23,30 +23,12 @@ jobs:
         with:
           fetch-depth: 0
 
-      - uses: actions/download-artifact@v4
+      - name: download binaries
+        uses: ./.github/actions/download-binaries
+      - name: persist metrics
+        uses: ./.github/actions/persist metrics
         with:
-          name: binary-windows
-      - uses: actions/download-artifact@v4
-        with:
-          name: binary-darwin
-      - uses: actions/download-artifact@v4
-        with:
-          name: binary-linux
-
-      - name: set git user
-        uses: ./.github/actions/git-user
-      - name: make git-metrics executable
-        run: chmod +x ./git-metrics_linux-x86_64
-
-      - name: publish metrics
-        run: |
-          ./git-metrics_linux-x86_64 --backend command pull
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-pc-windows-msvc" --tag "platform.os: windows" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_win-x86_64.exe)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-pc-windows-msvc" --tag "platform.os: windows" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_win-aarch64.exe)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-apple-darwin" --tag "platform.os: macos" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_darwin-x86_64)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-apple-darwin" --tag "platform.os: macos" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_darwin-aarch64)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-unknown-linux-gnu" --tag "platform.os: linux" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_linux-x86_64)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-unknown-linux-gnu" --tag "platform.os: linux" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_linux-aarch64)
+          push: "false"
 
       - name: output git-metrics
         id: diff

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -39,31 +39,12 @@ jobs:
         with:
           fetch-depth: 0
 
-      - uses: actions/download-artifact@v4
+      - name: download binaries
+        uses: ./.github/actions/download-binaries
+      - name: persist metrics
+        uses: ./.github/actions/persist-metrics
         with:
-          name: binary-windows
-      - uses: actions/download-artifact@v4
-        with:
-          name: binary-darwin
-      - uses: actions/download-artifact@v4
-        with:
-          name: binary-linux
-
-      - name: set git user
-        uses: ./.github/actions/git-user
-      - name: make git-metrics executable
-        run: chmod +x ./git-metrics_linux-x86_64
-
-      - name: publish metrics
-        run: |
-          ./git-metrics_linux-x86_64 --backend command pull
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-pc-windows-msvc" --tag "platform.os: windows" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_win-x86_64.exe)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-pc-windows-msvc" --tag "platform.os: windows" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_win-aarch64.exe)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-apple-darwin" --tag "platform.os: macos" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_darwin-x86_64)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-apple-darwin" --tag "platform.os: macos" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_darwin-aarch64)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: x86_64-unknown-linux-gnu" --tag "platform.os: linux" --tag "platform.arch: x86_64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_linux-x86_64)
-          ./git-metrics_linux-x86_64 add binary-size --tag "build.target: aarch64-unknown-linux-gnu" --tag "platform.os: linux" --tag "platform.arch: aarch64" --tag "unit: byte" $(stat --printf="%s" ./git-metrics_linux-aarch64)
-          ./git-metrics_linux-x86_64 --backend command push
+          push: "true"
 
   publish:
     name: publish
@@ -72,15 +53,9 @@ jobs:
     needs:
       - execute-build
     steps:
-      - uses: actions/download-artifact@v4
-        with:
-          name: binary-windows
-      - uses: actions/download-artifact@v4
-        with:
-          name: binary-darwin
-      - uses: actions/download-artifact@v4
-        with:
-          name: binary-linux
+      - name: download binaries
+        uses: ./.github/actions/download-binaries
+
       - name: get release id from tag
         id: release_id
         run: |