Skip to content

fix: dependabot workerd PR skip description validation label#14232

Merged
NuroDev merged 1 commit into
mainfrom
fix/ci-labels
Jun 9, 2026
Merged

fix: dependabot workerd PR skip description validation label#14232
NuroDev merged 1 commit into
mainfrom
fix/ci-labels

Conversation

@NuroDev

@NuroDev NuroDev commented Jun 9, 2026

Copy link
Copy Markdown
Member

Fixes a bug with dependabot CI workflow file where the recent GitHub repository label changes broke it since it had the incorrect label.

  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because: CI workflow fix
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: Internal CI workflow fix

A picture of a cute animal (not mandatory, but encouraged)

@NuroDev NuroDev self-assigned this Jun 9, 2026
@changeset-bot

changeset-bot Bot commented Jun 9, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5254b04

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 Jun 9, 2026
@ask-bonk

ask-bonk Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

The change is correct and consistent. The C3 dependabot config at line 25 already used the prefixed label ci:skip-pr-description-validation, and this PR fixes the workerd/Miniflare config at line 45 to match the actual label checked by validate-pr-description.ts:37. The fix is accurate, minimal, and internally consistent.

LGTM

github run

@NuroDev NuroDev mentioned this pull request Jun 9, 2026
Merged
@NuroDev NuroDev added the ci:no-changeset-required Skip pull request checks for a valid changeset label Jun 9, 2026
@NuroDev NuroDev marked this pull request as ready for review June 9, 2026 09:41
@NuroDev NuroDev requested a review from workers-devprod as a code owner June 9, 2026 09:41
@workers-devprod workers-devprod requested review from a team and james-elicx and removed request for a team June 9, 2026 09:42
@workers-devprod

workers-devprod commented Jun 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

devin-ai-integration[bot]
devin-ai-integration Bot reviewed Jun 9, 2026
View reviewed changes

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

Copy link
Copy Markdown
Contributor

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

pkg-pr-new Bot commented Jun 9, 2026

Copy link
Copy Markdown
create-cloudflare

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

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@14232

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@14232

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

@cloudflare/wrangler-bundler

npm i https://pkg.pr.new/@cloudflare/wrangler-bundler@14232

commit: 5254b04

workers-devprod
workers-devprod approved these changes Jun 9, 2026
View reviewed changes

@workers-devprod workers-devprod left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Codeowners reviews satisfied

@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Jun 9, 2026
@NuroDev NuroDev merged commit 258357b into main Jun 9, 2026
67 of 70 checks passed
@NuroDev NuroDev deleted the fix/ci-labels branch June 9, 2026 10:02
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments
@james-elicx james-elicx james-elicx approved these changes
@workers-devprod workers-devprod workers-devprod approved these changes

Assignees

@NuroDev NuroDev

Labels

ci:no-changeset-required Skip pull request checks for a valid changeset

Projects

workers-sdk
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@NuroDev @workers-devprod @james-elicx