Skip to content

Commit 4c06e91

Browse files
committed
[refactor] Tests running on every PR and pushes to main and release
branches * Instead of calling it from verification workflows Signed-off-by: dd di cesare <[email protected]>
1 parent 8384d2a commit 4c06e91

File tree

3 files changed

+8
-12
lines changed

3 files changed

+8
-12
lines changed

.github/workflows/test.yaml

+6-1
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,13 @@
11
name: Test
22

33
on:
4-
workflow_call:
54
workflow_dispatch:
5+
push:
6+
branches: ['main', 'release-v*']
7+
pull_request:
8+
branches: ['*']
9+
schedule:
10+
- cron: "15 1 * * *"
611

712
jobs:
813
pre-job:

.github/workflows/verify-main.yaml

+1-5
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,6 @@ jobs:
2424
with:
2525
cancel_others: false
2626
paths_ignore: '["**.adoc", "**.md", "examples/**", "LICENSE"]'
27-
test:
28-
name: Calls test workflow
29-
uses: ./.github/workflows/test.yaml
30-
secrets: inherit
3127

3228
verify-manifests:
3329
name: Validate set of manifests
@@ -55,7 +51,7 @@ jobs:
5551
# This check adds a list of checks to one job to simplify adding settings to the repo.
5652
# If a new check is added in this file, and it should be retested on entry to the merge queue,
5753
# it needs to be added to the list below aka needs: [ existing check 1, existing check 2, new check ].
58-
needs: [ test, verify-manifests ]
54+
needs: [ verify-manifests ]
5955
if: always()
6056
runs-on: ubuntu-latest
6157
steps:

.github/workflows/verify-release.yaml

+1-6
Original file line numberDiff line numberDiff line change
@@ -23,11 +23,6 @@ jobs:
2323
cancel_others: false
2424
paths_ignore: '["**.adoc", "**.md", "examples/**", "LICENSE"]'
2525

26-
test:
27-
name: Calls test workflow
28-
uses: ./.github/workflows/test.yaml
29-
secrets: inherit
30-
3126
verify-release:
3227
name: Validate release manifests
3328
if: needs.pre-job.outputs.should_skip != 'true'
@@ -53,7 +48,7 @@ jobs:
5348
# This check adds a list of checks to one job to simplify adding settings to the repo.
5449
# If a new check is added in this file, and it should be retested on entry to the merge queue,
5550
# it needs to be added to the list below aka needs: [ existing check 1, existing check 2, new check ].
56-
needs: [ test, verify-release ]
51+
needs: [ verify-release ]
5752
if: always()
5853
runs-on: ubuntu-latest
5954
steps:

0 commit comments

Comments
 (0)