-
Notifications
You must be signed in to change notification settings - Fork 10
docs(mq): add scopes #9467
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(mq): add scopes #9467
Conversation
This adds a general documentation page on the scope features for itself. Change-Id: I87095c316b9a9995f59cf54ea29d5d7955dd477e
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 🤖 Continuous IntegrationWonderful, this rule succeeded.
🟢 👀 Review RequirementsWonderful, this rule succeeded.
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 ReviewsWonderful, this rule succeeded.
🟢 📕 PR descriptionWonderful, this rule succeeded.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds comprehensive documentation for Mergify's scopes feature, which enables intelligent batching of pull requests based on the areas of code they modify. The documentation explains how scope-aware batching optimizes CI resource usage by grouping related changes together, particularly beneficial for monorepo environments.
Key changes:
- New dedicated documentation page explaining scopes, their lifecycle, configuration, and best practices
- Navigation menu updated to include the new Scopes page
- Cross-references added from existing merge queue and monorepo documentation
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/content/navItems.tsx | Adds "Scopes" navigation item with FaLayerGroup icon between Batches and Two-Step CI |
| src/content/docs/merge-queue/scopes.mdx | New comprehensive documentation page covering scope-aware batching, configuration, and usage |
| src/content/docs/merge-queue/monorepo.mdx | Adds cross-reference link to the new scopes documentation |
| src/content/docs/merge-queue.mdx | Adds scope-aware batching as a cost optimization strategy with proper formatting |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Merge Queue Status
🚫 The pull request has left the queue (rule: This pull request spent 28 seconds in the queue, including 14 seconds waiting for CI. Required conditions to merge
ReasonThe merge conditions cannot be satisfied due to failing checks Failing checks:
HintYou may have to fix your CI before adding the pull request to the queue again. |
Merge Queue Status
🚫 The pull request has left the queue (rule: This pull request spent 28 seconds in the queue, with no time waiting for CI. Required conditions to merge
ReasonThe merge conditions cannot be satisfied due to failing checks Failing checks:
HintYou may have to fix your CI before adding the pull request to the queue again. |
|
@Mergifyio requeue |
✅ This pull request will be re-embarked automaticallyThe followup |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at 214706c |
Merge Queue Status
✅ The pull request has been merged This pull request spent 2 minutes 26 seconds in the queue, including 1 minute 48 seconds waiting for CI. Required conditions to merge
|
This adds a general documentation page on the scope features for itself.