Skip to content

[devprod-status-bot] Simplify Version Packages PR CI failure alerts#12435

Merged
petebacondarwin merged 1 commit intomainfrom
pbacondarwin/simplify-version-packages-ci-alerts
Feb 5, 2026
Merged

[devprod-status-bot] Simplify Version Packages PR CI failure alerts#12435
petebacondarwin merged 1 commit intomainfrom
pbacondarwin/simplify-version-packages-ci-alerts

Conversation

@petebacondarwin
Copy link
Copy Markdown
Contributor

@petebacondarwin petebacondarwin commented Feb 5, 2026

Simplify the Version Packages PR CI failure alert logic by removing the required status checks fetch from GitHub's branch protection API.

The bot now sends an alert for any failing CI job on the Version Packages PR, instead of first fetching the required status checks and filtering. This removes unnecessary complexity and ensures all CI failures are reported.

Changes:

  • Removed RequiredStatusChecksResult type
  • Removed getRequiredStatusChecks() function
  • Removed sendGitHubAPIFailureAlert() function
  • Simplified the CI failure handling logic

  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because: This is a simplification that removes code paths; the remaining logic is straightforward
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: Internal tooling change

Open with Devin

@petebacondarwin petebacondarwin requested a review from a team as a code owner February 5, 2026 17:57
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Feb 5, 2026

🦋 Changeset detected

Latest commit: 0f953fd

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Feb 5, 2026

Claude finished @petebacondarwin's task —— View job


Changeset Review

Todo List

  • Read changeset guidelines from .changeset/README.md
  • Read the changeset file .changeset/simplify-version-packages-ci-alerts.md
  • Validate changeset against criteria
  • Report findings

Results

All changesets look good

The changeset for @cloudflare/devprod-status-bot is well-formatted and appropriate:

  • Version type: patch is correct for this internal tooling simplification
  • Description: Clear and meaningful - explains what changed and why
  • No markdown headers: Properly formatted without h1/h2/h3 headers
  • Package scope: Correctly targets the affected package

@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Feb 5, 2026
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Feb 5, 2026

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@12435

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@12435

miniflare

npm i https://pkg.pr.new/miniflare@12435

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@12435

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@12435

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@12435

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@12435

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@12435

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@12435

wrangler

npm i https://pkg.pr.new/wrangler@12435

commit: 0f953fd

@petebacondarwin petebacondarwin merged commit c2163df into main Feb 5, 2026
42 of 44 checks passed
@petebacondarwin petebacondarwin deleted the pbacondarwin/simplify-version-packages-ci-alerts branch February 5, 2026 18:19
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants