From 576e86b3b6106ae139f504dfd0e360ee1510e373 Mon Sep 17 00:00:00 2001
From: Jackson Burns <33505528+JacksonBurns@users.noreply.github.com>
Date: Wed, 5 Jun 2024 10:26:21 -0400
Subject: [PATCH] try bumping upload-artifact version, switch to official
 download-artifact

---
 .github/workflows/CI.yml | 20 +++++++-------------
 1 file changed, 7 insertions(+), 13 deletions(-)

diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
index 75deef0616..e3ac04c872 100644
--- a/.github/workflows/CI.yml
+++ b/.github/workflows/CI.yml
@@ -185,7 +185,7 @@ jobs:
       # Upload Regression Results as Failed if above step failed
       - name: Upload Failed Results
         if: ${{ failure() && steps.regression-execution.conclusion == 'failure' }}
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: failed_regression_results
           path: |
@@ -195,7 +195,7 @@ jobs:
       - name: Upload Results as Reference
         # upload the results for scheduled CI (on main) and pushes to main
         if: ${{ env.REFERENCE_JOB == 'true' }}
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: stable_regression_results
           path: |
@@ -204,7 +204,7 @@ jobs:
       # Upload Regression Results as Dynamic if Push to non-main Branch
       - name: Upload Results as Dynamic
         if: ${{ env.REFERENCE_JOB == 'false' }}
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: dynamic_regression_results
           path: |
@@ -215,23 +215,17 @@ jobs:
         run: mkdir stable_regression_results
 
       # Retrieve Stable Results for reference
-      # Will need to use this -> https://github.com/dawidd6/action-download-artifact
       - name: Retrieve Stable Regression Results
         if: ${{ env.REFERENCE_JOB == 'false' }}
-        uses: dsnopek/action-download-artifact@91dda23aa09c68860977dd0ed11d93c0ed3795e7 # see https://github.com/ReactionMechanismGenerator/RMG-Py/pull/2459#issuecomment-1582850815
+        uses: actions/download-artifact@v4
         with:
         # this will search for the last successful execution of CI on main and download
         # the stable regression results
-          workflow: CI.yml
-          workflow_conclusion: success
-          repo: ReactionMechanismGenerator/RMG-Py
-          branch: main
+          # run-id: need this?
+          repository: ReactionMechanismGenerator/RMG-Py
+          github-token:  ${{ secrets.GH_PAT }}
           name: stable_regression_results
           path: stable_regression_results
-          search_artifacts: true  # retrieves the last run result, either scheduled daily or on push to main
-          ensure_latest: true     # ensures that the latest run is retrieved
-          # should result in a set of folders inside stable_regression_results
-          # each of which has the stable result for that example/test
 
       # Regression Testing - Actual Comparisons
       - name: Regression Tests - Compare to Baseline