From 5a12c97d9fcac8ddca7c308b55d6c4a4fc53b2f2 Mon Sep 17 00:00:00 2001 From: Richard West Date: Wed, 2 Aug 2023 14:22:01 -0400 Subject: [PATCH 1/2] Fix CI workflow to function on forked pushes. This is what we did in the (abandoned) PR #2510 and should have been done in PR #2512. It means that when the CI runs on pushes to a forked repository, it checks out that forked repository. --- .github/workflows/CI.yml | 4 ---- 1 file changed, 4 deletions(-) diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml index 251fa9cd4f..3fad8b11d0 100644 --- a/.github/workflows/CI.yml +++ b/.github/workflows/CI.yml @@ -62,8 +62,6 @@ jobs: steps: - name: Checkout RMG-Py uses: actions/checkout@v3 - with: - repository: ReactionMechanismGenerator/RMG-Py # configures the mamba environment manager and builds the environment - name: Setup Mambaforge Python 3.7 @@ -114,8 +112,6 @@ jobs: steps: - name: Checkout RMG-Py uses: actions/checkout@v3 - with: - repository: ReactionMechanismGenerator/RMG-Py # configures the mamba environment manager and builds the environment - name: Setup Mambaforge Python 3.7 From 5008b8892a1ba68bc4cac4958f14f4e123cd7286 Mon Sep 17 00:00:00 2001 From: Richard West Date: Wed, 2 Aug 2023 19:54:08 -0400 Subject: [PATCH 2/2] Update header comments in CI workflow file. I still question the utility of a changelog when we have the git history, but I've updated it anyway. And removed the description of things that were removed a few days ago. --- .github/workflows/CI.yml | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml index 3fad8b11d0..f13f4cdb54 100644 --- a/.github/workflows/CI.yml +++ b/.github/workflows/CI.yml @@ -12,10 +12,6 @@ # them. These 'answers' are re-generated daily, or on any push to main, and retrieved whenever # a push is made to a non-main branch. The new proposed changes are referred to as "Dynamic". # -# The `workflow_call` section below allows running this exact file from RMG-database, saving -# us from having to maintain two copies of it. The only liens needed to facilitate that are -# the `workflow_call` block in the `on` section and the passed `rmg-db-branch` so that we can -# pull appropriately for RMG-database. # # Changelog: # 2023-04 - Jackson Burns - Added this header, regression tests, cleanup of action in @@ -28,6 +24,8 @@ # 2023-06-27 - add option to run from RMG-database with GitHub resuable workflows # 2023-07-17 - made it pass by default # 2023-07-21 - upload the regression results summary as artifact (for use as a comment on PRs) +# 2023-07-31 - removed option to run from RMG-database with GitHub resuable workflows + name: Continuous Integration on: