Skip to content

Commit 94535c1

Browse files
chattontac0turtle
andauthored
fix: remove build on merge groups (#2354)
<!-- Please read and fill out this form before submitting your PR. Please make sure you have reviewed our contributors guide before submitting your first PR. NOTE: PR titles should follow semantic commits: https://www.conventionalcommits.org/en/v1.0.0/ --> ## Overview We were building images during merge_group events as well, we don't need to build that many images so I think it's safe to skip. <!-- Please provide an explanation of the PR, including the appropriate context, background, goal, and rationale. If there is an issue with this information, please provide a tl;dr and link the issue. Ex: Closes #<issue number> --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Chores** - Updated workflow to skip building Docker images for merge group events, reducing unnecessary builds. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Marko <marko@baricevic.me>
1 parent 4ea5871 commit 94535c1

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

.github/workflows/test.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@ on:
1111
jobs:
1212
build-docker-image:
1313
name: Build Docker Image
14+
# skip building images for merge groups as they are already built on PRs and main
15+
if: github.event_name != 'merge_group'
1416
runs-on: ubuntu-latest
1517
steps:
1618
- name: Checkout code

0 commit comments

Comments
 (0)