Skip to content

CI - Make new CI checks run in merge queue#3016

Merged
bfops merged 4 commits into
masterfrom
bfops/merge-group-ci
Aug 1, 2025
Merged

CI - Make new CI checks run in merge queue#3016
bfops merged 4 commits into
masterfrom
bfops/merge-group-ci

Conversation

@bfops
Copy link
Copy Markdown
Collaborator

@bfops bfops commented Aug 1, 2025

Description of Changes

A few of our new CI checks weren't set up to run in the merge queue, which prevented PRs from merging when those checks were marked required.

API and ABI breaking changes

None. CI only change.

Expected complexity level and risk

1

Testing

This will have to be tested by making these checks required again, and then seeing if this PR can merge.

@bfops bfops requested a review from jdetter August 1, 2025 17:04
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml
Copy link
Copy Markdown
Contributor

@jdetter jdetter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with comment

Comment thread .github/workflows/ci.yml Fixed
@bfops bfops added this pull request to the merge queue Aug 1, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 1, 2025
@bfops bfops added this pull request to the merge queue Aug 1, 2025
Merged via the queue into master with commit f664a63 Aug 1, 2025
25 checks passed
@bfops bfops deleted the bfops/merge-group-ci branch August 19, 2025 21:07
mamcx pushed a commit that referenced this pull request Aug 26, 2025
# Description of Changes

A few of our new CI checks weren't set up to run in the merge queue,
which prevented PRs from merging when those checks were marked required.

# API and ABI breaking changes

None. CI only change.

# Expected complexity level and risk

1

# Testing

This will have to be tested by making these checks required again, and
then seeing if this PR can merge.

---------

Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants