Skip to content

Conversation

@JacobCoffee
Copy link
Member

Description

  • In Deduplicate CI spam #2852 it seems it caused some checks to not run on merge on main which may or may not affect cabotage deploys

@JacobCoffee JacobCoffee requested a review from ewdurbin as a code owner January 13, 2026 20:57
@JacobCoffee JacobCoffee requested a review from hugovk January 13, 2026 20:57
@ewdurbin
Copy link
Member

tl;dr: Cabotage was correctly handling all hooks, but creation of the deployment failed because by default all required checks on the repo must pass. We have test as a required check, so after getting the web hook for a merge to main, it was unable to create a GitHub deployment object since test was skipped.

@JacobCoffee JacobCoffee merged commit 848bab1 into main Jan 13, 2026
10 checks passed
@JacobCoffee JacobCoffee deleted the fix-ci-checks branch January 13, 2026 21:20
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.

4 participants