Skip to content

Commit

Permalink
workflow: run tests as part of the merge group
Browse files Browse the repository at this point in the history
This commit adds the missing `merge_group` to the pytest and go
test workflows.

Note that I added `types: [ "checks_requested" ]` (just like we
have for the container.yaml) but I'm not actually sure about it,
the GH docs are a bit unclear to me here, it seems they suggest
to keep this generic but then the example explicitly uses `type`
(I'm probably overthinking this fwiw).

[1] https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#merge_group
  • Loading branch information
mvo5 authored and ondrejbudai committed Dec 19, 2024
1 parent f89342a commit 119e8fc
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/go.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ on:
branches: [ "main" ]
pull_request:
branches: [ "main" ]
merge_group:
types: [ "checks_requested" ]

jobs:

Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/pytest.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ on:
branches: [ "main" ]
pull_request:
branches: [ "main" ]
merge_group:
types: [ "checks_requested" ]

jobs:
build:
Expand Down

0 comments on commit 119e8fc

Please sign in to comment.