Skip to content

Commit 5ed1b5d

Browse files
committed
Remove unnecessary GitHub Action triggers
This goes from `[push, pull_request]` to just `push` on all four GitHub Actions. I also added `paths` elements to each GitHub Action that will hopefully reduce the amount of unnecessary work in GitHub Actions. I also added `on: workflow_dispatch` so we should be able to manually trigger each of these actions if necessary.
1 parent ac9b283 commit 5ed1b5d

File tree

4 files changed

+42
-12
lines changed

4 files changed

+42
-12
lines changed

.github/workflows/arraymerge_gtest.yml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,17 @@
11
name: array-merge-gtest
22

3-
on: [push, pull_request]
4-
# paths:
5-
# - 'array_merge/**'
3+
on: push
4+
paths:
5+
- 'array_merge/**'
6+
# We probably want to re-run the checks if the definition of
7+
# `mergesort` changes since this depends on that.
8+
- 'mergesort/mergesort.[ch]'
9+
10+
# This lets us manually trigger this action
11+
# through the GitHub web interface. See
12+
# https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_dispatch
13+
# for more details.
14+
on: workflow_dispatch
615

716
jobs:
817
build:

.github/workflows/arraymerge_test_valgrind.yml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,17 @@
11
name: array-merge-test-valgrind
22

3-
on: [push, pull_request]
4-
# paths:
5-
# - 'array_merge/**'
3+
on: push
4+
paths:
5+
- 'array_merge/**'
6+
# We probably want to re-run the checks if the definition of
7+
# `mergesort` changes since this depends on that.
8+
- 'mergesort/mergesort.[ch]'
9+
10+
# This lets us manually trigger this action
11+
# through the GitHub web interface. See
12+
# https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_dispatch
13+
# for more details.
14+
on: workflow_dispatch
615

716
jobs:
817
build:

.github/workflows/mergesort_gtest.yml

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,14 @@
11
name: mergesort-gtest
22

3-
on: [push, pull_request]
4-
# paths:
5-
# - 'mergesort/**'
3+
on: push
4+
paths:
5+
- 'mergesort/**'
6+
7+
# This lets us manually trigger this action
8+
# through the GitHub web interface. See
9+
# https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_dispatch
10+
# for more details.
11+
on: workflow_dispatch
612

713
jobs:
814
build:

.github/workflows/mergesort_test_valgrind.yml

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,14 @@
11
name: mergesort-test-valgrind
22

3-
on: [push, pull_request]
4-
# paths:
5-
# - 'mergesort/**'
3+
on: push
4+
paths:
5+
- 'mergesort/**'
6+
7+
# This lets us manually trigger this action
8+
# through the GitHub web interface. See
9+
# https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_dispatch
10+
# for more details.
11+
on: workflow_dispatch
612

713
jobs:
814
build:

0 commit comments

Comments
 (0)