From b26acea6777c762cbe224ebf782186ca9a8ad511 Mon Sep 17 00:00:00 2001
From: Alexander Entinger <cto@lxrobotics.com>
Date: Wed, 21 Feb 2024 07:06:18 +0100
Subject: [PATCH] Fix regression: report size delta size on PR. (#63)

The necessary steps have in fact been documented here: https://github.com/arduino/report-size-deltas/blob/main/docs/FAQ.md#workflow-triggered-by-pull_request-event but I have overlooked them when I fixed the upload issue. With this PR the size deltas are - once again - reported within the PR.
---
 .github/workflows/compile-examples.yml   | 27 ++++++++++++++++++++----
 .github/workflows/report-size-deltas.yml | 21 ------------------
 .gitignore                               |  1 +
 3 files changed, 24 insertions(+), 25 deletions(-)
 delete mode 100644 .github/workflows/report-size-deltas.yml
 create mode 100644 .gitignore

diff --git a/.github/workflows/compile-examples.yml b/.github/workflows/compile-examples.yml
index 25f7204..ff8d550 100644
--- a/.github/workflows/compile-examples.yml
+++ b/.github/workflows/compile-examples.yml
@@ -20,13 +20,14 @@ on:
   # See: https://docs.github.com/en/free-pro-team@latest/actions/reference/events-that-trigger-workflows#repository_dispatch
   repository_dispatch:
 
+env:
+  # It's convenient to set variables for values used multiple times in the workflow.
+  SKETCHES_REPORTS_PATH: sketches-reports
+
 jobs:
-  build:
+  compile:
     runs-on: ubuntu-latest
 
-    env:
-      SKETCHES_REPORTS_PATH: sketches-reports
-
     strategy:
       fail-fast: false
 
@@ -83,3 +84,21 @@ jobs:
         with:
           name: sketches-report-${{ matrix.board.artifact-name-suffix }}
           path: ${{ env.SKETCHES_REPORTS_PATH }}
+
+  # When using a matrix to compile for multiple boards, it's necessary to use a separate job for the deltas report
+  report:
+    needs: compile # Wait for the compile job to finish to get the data for the report
+    if: github.event_name == 'pull_request' # Only run the job when the workflow is triggered by a pull request
+    runs-on: ubuntu-latest
+
+    steps:
+      # This step is needed to get the size data produced by the compile jobs
+      - name: Download sketches reports artifacts
+        uses: actions/download-artifact@v4
+        with:
+          # All workflow artifacts will be downloaded to this location.
+          path: ${{ env.SKETCHES_REPORTS_PATH }}
+
+      - uses: arduino/report-size-deltas@v1
+        with:
+          sketches-reports-source: ${{ env.SKETCHES_REPORTS_PATH }}
diff --git a/.github/workflows/report-size-deltas.yml b/.github/workflows/report-size-deltas.yml
deleted file mode 100644
index c69650f..0000000
--- a/.github/workflows/report-size-deltas.yml
+++ /dev/null
@@ -1,21 +0,0 @@
-name: Report Size Deltas
-
-on:
-  schedule:
-    - cron: '*/5 * * * *'
-  # See: https://docs.github.com/en/free-pro-team@latest/actions/reference/events-that-trigger-workflows#workflow_dispatch
-  workflow_dispatch:
-  # See: https://docs.github.com/en/free-pro-team@latest/actions/reference/events-that-trigger-workflows#repository_dispatch
-  repository_dispatch:
-
-jobs:
-  report:
-    runs-on: ubuntu-latest
-
-    steps:
-      # See: https://github.com/arduino/actions/blob/master/libraries/report-size-deltas/README.md
-      - name: Comment size deltas reports to PRs
-        uses: arduino/report-size-deltas@main
-        with:
-          # The name of the workflow artifact created by the "Compile Examples" workflow
-          sketches-reports-source: sketches-reports
diff --git a/.gitignore b/.gitignore
new file mode 100644
index 0000000..9f11b75
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1 @@
+.idea/