Skip to content

ci: skip Bonk PR review on Version Packages PRs#13934

Open
petebacondarwin wants to merge 1 commit into
mainfrom
skip-bonk-on-version-packages
Open

ci: skip Bonk PR review on Version Packages PRs#13934
petebacondarwin wants to merge 1 commit into
mainfrom
skip-bonk-on-version-packages

Conversation

@petebacondarwin
Copy link
Copy Markdown
Contributor

@petebacondarwin petebacondarwin commented May 15, 2026

Skips the Bonk PR review workflow on the auto-generated "Version Packages" PRs created by the changesets action. These PRs only bump versions and update changelogs, so they don't need an AI review.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because: this is a workflow if condition change that will be exercised the next time a Version Packages PR is opened.
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: this is an internal CI workflow change with no user-facing impact.

Open in Devin Review

@petebacondarwin petebacondarwin added no-changeset-required Skip PR check for valid changeset ci-flake Applied to PRs addressing CI flakiness labels May 15, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 15, 2026

⚠️ No Changeset found

Latest commit: a2898f9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@github-project-automation github-project-automation Bot moved this to Untriaged in workers-sdk May 15, 2026
@workers-devprod workers-devprod requested review from a team and NuroDev and removed request for a team May 15, 2026 15:21
@workers-devprod
Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • @cloudflare/wrangler
Show detailed file reviewers
  • .github/workflows/bonk-pr-review.yml: [@cloudflare/wrangler]

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 15, 2026

LGTM

github run

Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

LGTM. The condition correctly identifies the changesets "Version Packages" PR by matching the changeset-release/main head branch within the cloudflare org, and the multi-line if with the explanatory comment is clear.

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 15, 2026

Approved PR #13934. The change adds a simple and correct condition to skip the Bonk PR review workflow on "Version Packages" PRs by checking for the changeset-release/main head branch in the cloudflare org. No issues found.

github run

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 1 additional finding.

Open in Devin Review

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 15, 2026

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: a2898f9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-flake Applied to PRs addressing CI flakiness no-changeset-required Skip PR check for valid changeset

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

2 participants