Skip to content

docs(merge-queue): remove partition rules note from merge-batch strategy#11691

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/sileht/docs-remove-partition-merge-strategies/remove-partition-rules-note-merge-batch-strategy--3298a079
Jun 1, 2026
Merged

docs(merge-queue): remove partition rules note from merge-batch strategy#11691
mergify[bot] merged 1 commit into
mainfrom
devs/sileht/docs-remove-partition-merge-strategies/remove-partition-rules-note-merge-batch-strategy--3298a079

Conversation

@sileht
Copy link
Copy Markdown
Member

@sileht sileht commented Jun 1, 2026

Partitions are being deprecated in favor of scopes. The merge-batch
strategy is compatible with scopes (only partition_rules trigger the
engine's incompatibility check), so the note no longer applies and is
removed rather than re-pointed at scopes.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

Copilot AI review requested due to automatic review settings June 1, 2026 12:09
@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 1, 2026 12:09 Failure
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Jun 1, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes an outdated note from the Merge Queue “merge-batch” strategy documentation now that partitions are being deprecated in favor of scopes.

Changes:

  • Removed the “Not compatible with partition rules” constraint from the merge-batch strategy section.
  • Added a .plan-prompt.txt file containing local/agent planning instructions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/content/docs/merge-queue/merge-strategies.mdx Drops an outdated/incorrect compatibility note from the merge-batch strategy constraints.
.plan-prompt.txt Introduces a local planning prompt file (should likely not be committed).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .plan-prompt.txt Outdated
@sileht sileht force-pushed the devs/sileht/docs-remove-partition-merge-strategies/remove-partition-rules-note-merge-batch-strategy--3298a079 branch from 16e6fa0 to 8417ed9 Compare June 1, 2026 12:13
@sileht
Copy link
Copy Markdown
Member Author

sileht commented Jun 1, 2026

Revision history

# Type Changes Reason Date
1 initial 16e6fa0 2026-06-01 12:13 UTC
2 content 16e6fa0 → 8417ed9 (raw) 2026-06-01 12:13 UTC
3 content 8417ed9 → 3332cba (raw) 2026-06-01 12:51 UTC

@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 1, 2026 12:14 Failure
@mergify mergify Bot requested a review from a team June 1, 2026 12:16
Comment thread .gitignore Outdated
Comment thread src/content/docs/merge-queue/merge-strategies.mdx
Partitions are being deprecated in favor of scopes. The merge-batch
strategy is compatible with scopes (only partition_rules trigger the
engine's incompatibility check), so the note no longer applies and is
removed rather than re-pointed at scopes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Change-Id: I3298a079529eae3b2d08828c1e81b2b42f6f2455
@sileht sileht force-pushed the devs/sileht/docs-remove-partition-merge-strategies/remove-partition-rules-note-merge-batch-strategy--3298a079 branch from 8417ed9 to 3332cba Compare June 1, 2026 12:51
@mergify mergify Bot deployed to Mergify Merge Protections June 1, 2026 12:51 Active
@sileht sileht marked this pull request as ready for review June 1, 2026 13:19
@mergify mergify Bot requested a review from a team June 1, 2026 13:32
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Jun 1, 2026

Merge Queue Status

This pull request spent 2 minutes 40 seconds in the queue, including 1 minute 54 seconds running CI.

Required conditions to merge

mergify Bot added a commit that referenced this pull request Jun 1, 2026
@mergify mergify Bot added the queued label Jun 1, 2026
@mergify mergify Bot merged commit 7310697 into main Jun 1, 2026
8 of 13 checks passed
@mergify mergify Bot deleted the devs/sileht/docs-remove-partition-merge-strategies/remove-partition-rules-note-merge-batch-strategy--3298a079 branch June 1, 2026 13:38
@mergify mergify Bot removed the queued label Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants