Skip to content

Allow GitLab docs merge requests in PR validation - #15691

Merged
tpmmorris merged 1 commit into
mainfrom
tpmmorris/fix/allow-gitlab-docs
Sep 17, 2026
Merged

tpmmorris merged 1 commit into
mainfrom
tpmmorris/fix/allow-gitlab-docs

Conversation

@tpmmorris

@tpmmorris tpmmorris commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

The PR description validator currently only allows PRs to the GitHub cloudflare-docs repository. We should also allow MRs to the GitLab repository, as they are mirrored.


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

Devin Review

@changeset-bot

changeset-bot Bot commented Sep 17, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 544bc01

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 Sep 17, 2026
@workers-devprod
workers-devprod requested review from a team and NuroDev and removed request for a team September 17, 2026 10:47
@workers-devprod

workers-devprod commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@ask-bonk

ask-bonk Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

@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 found 1 potential issue.

Devin Review

Comment thread tools/deployments/validate-pr-description.ts
@pkg-pr-new

pkg-pr-new Bot commented Sep 17, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@15691

@cloudflare/build-output-utils

npm i https://pkg.pr.new/@cloudflare/build-output-utils@15691

@cloudflare/codemods

npm i https://pkg.pr.new/@cloudflare/codemods@15691

@cloudflare/config

npm i https://pkg.pr.new/@cloudflare/config@15691

@cloudflare/containers-shared

npm i https://pkg.pr.new/@cloudflare/containers-shared@15691

create-cloudflare

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

@cloudflare/deploy-helpers

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-functions

npm i https://pkg.pr.new/@cloudflare/pages-functions@15691

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-plugin

npm i https://pkg.pr.new/@cloudflare/vitest-plugin@15691

@cloudflare/workers-auth

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: 544bc01

@tpmmorris tpmmorris added the ci:no-changeset-required Skip pull request checks for a valid changeset label Sep 17, 2026
@NuroDev
NuroDev enabled auto-merge (squash) September 17, 2026 13:49
@tpmmorris
tpmmorris disabled auto-merge September 17, 2026 13:49

@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 Sep 17, 2026
@tpmmorris
tpmmorris enabled auto-merge (squash) September 17, 2026 13:51
@tpmmorris
tpmmorris force-pushed the tpmmorris/fix/allow-gitlab-docs branch from f8459ed to 544bc01 Compare September 17, 2026 15:20
@tpmmorris
tpmmorris merged commit 99044e5 into main Sep 17, 2026
55 checks passed
@tpmmorris
tpmmorris deleted the tpmmorris/fix/allow-gitlab-docs branch September 17, 2026 15:49
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants