Skip to content

Integration merge queue - #1261

Merged
Joao-Dionisio merged 4 commits into
masterfrom
integration-merge-queue
Oct 8, 2026
Merged

Joao-Dionisio merged 4 commits into
masterfrom
integration-merge-queue

Conversation

@Joao-Dionisio

Copy link
Copy Markdown
Member

An attempt to fix #1249

@MySweetEden , can you please try to take a look at this?

@MySweetEden

Copy link
Copy Markdown
Contributor

Thanks for creating this PR. I wasn’t sure how to set up merge_group, so I couldn’t create the PR myself. 😂 I’ve reviewed the PR, and the changes match what I had in mind.

Do we also need to require the merge queue for master and add integration-gate as a required status check?

I also have one question about the current setup. I noticed that the pytest step in MacOS-test is commented out. Since the goal is to catch platform-specific test failures before merging, is there a reason why it is disabled?

@Joao-Dionisio

Copy link
Copy Markdown
Member Author

Do we also need to require the merge queue for master and add integration-gate as a required status check?

I think so, to run the tests at merge time.

is there a reason why it is disabled?

I don't quite remember, but I think it was either because it was too slow, or because it was constantly failing and we could not reproduce the error.

@Joao-Dionisio
Joao-Dionisio merged commit f202893 into master Oct 8, 2026
7 checks passed
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.

Run integration tests only at merge time, not on PR updates

2 participants