From c1c5147e530ce05e0f1c8f60dbb769c809f5194b Mon Sep 17 00:00:00 2001 From: "Bobby R. Bruce" Date: Thu, 10 Oct 2024 10:13:56 -0700 Subject: [PATCH] tests,misc: Remove `edited` from PR Action trigger list (#1654) `edited` is what forces a re-run of our tests when the PR title is updated and other minor metadata stuff. I believe all changes to the code are covered by the remainder. `synchronize` is means the PR is triggered with the when the this PR is from (in this case my forked gem5 repo) is synced with the PR branch here. This covers the vast majority of cases we care about. `opended` covers for the case where the PR is created and `ready_for_review` for when something moves out of a draft. --- .github/workflows/ci-tests.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci-tests.yaml b/.github/workflows/ci-tests.yaml index 4a4ecb5ba4..f40f3faff9 100644 --- a/.github/workflows/ci-tests.yaml +++ b/.github/workflows/ci-tests.yaml @@ -5,7 +5,7 @@ name: CI Tests on: pull_request: - types: [opened, edited, synchronize, ready_for_review] + types: [opened, synchronize, ready_for_review] concurrency: group: ${{ github.workflow }}-${{ github.ref || github.run_id }}