Skip to content

Conversation

@jacobweinstock
Copy link
Member

Description

Disable speculative checks and allow Mergify to merge PRs from me.

Why is this needed

Fixes: #

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

Speculative checks by default are enabled
by default now. We didn't use to have them.
We don't have the volume of PRs to need this.
It also adds significant time as we have to wait
for the CI of another PR to run.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
This way I can use the GitHub labels
to have mergify do the merging.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock jacobweinstock added the ready-to-merge Signal to Mergify to merge the PR. label Jul 31, 2025
@jacobweinstock jacobweinstock merged commit 1029df0 into tinkerbell:main Jul 31, 2025
24 of 28 checks passed
@jacobweinstock jacobweinstock deleted the update-mergifyio branch July 31, 2025 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Signal to Mergify to merge the PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant