-
Notifications
You must be signed in to change notification settings - Fork 10
docs: reference MGA #10112
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: reference MGA #10112
Conversation
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 streamlines the Merge Queue documentation by moving conceptual explanations to the external Merge Queue Academy and reshaping the in-product docs into an overview + feature/deep-dive structure.
Changes:
- Renames the Merge Queue nav item from “Introduction” to “Overview” and rewrites
/merge-queue.mdxinto a concise overview page with feature links and an Academy callout. - Adds Academy callouts/links across multiple Merge Queue docs (setup, two-step CI, performance, parallel checks, monorepo, batches) to point readers to deeper conceptual material on merge-queue.academy.
- Enhances the docs home page Merge Queue section with an AcademyCallout while keeping the existing feature grid.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
src/content/navItems.tsx |
Updates the Merge Queue child nav label to “Overview” to reflect the new overview page. |
src/content/docs/merge-queue/two-step.mdx |
Adds a tip callout linking to Merge Queue Academy’s two-step CI conceptual guide. |
src/content/docs/merge-queue/setup.mdx |
Adds a note directing users new to merge queues to Merge Queue Academy before setup. |
src/content/docs/merge-queue/performance.mdx |
Adds a tip linking to a high-velocity teams guide on Merge Queue Academy. |
src/content/docs/merge-queue/parallel-checks.mdx |
Adds a tip with a conceptual explanation link for speculative checks on Merge Queue Academy. |
src/content/docs/merge-queue/monorepo.mdx |
Adds a tip linking to Merge Queue Academy’s monorepo strategy content. |
src/content/docs/merge-queue/batches.mdx |
Adds a tip linking to Merge Queue Academy’s batching concepts page. |
src/content/docs/merge-queue.mdx |
Replaces the long conceptual introduction with a shorter overview, Academy callout, problem/benefit sections, key features table, and “learn more” links. |
src/content/docs/index.mdx |
Imports and uses AcademyCallout in the Merge Queue section; note that the Merge Queue docset card title is still “Introduction” while nav/page now use “Overview”. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
b4f3f52 to
58f50ca
Compare
|
This pull request is part of a stack:
|
This simplifies a bit the docs to move some of the MQ concepts outside of the docs. Change-Id: I02d2d065fd1d25f0a00d7d6801fad8b00987ff6d
58f50ca to
0b10033
Compare
Merge Queue Status✅ The pull request has been merged at 0b10033 This pull request spent 2 minutes 44 seconds in the queue, including 2 minutes 17 seconds running CI. Required conditions to merge
|
This simplifies a bit the docs to move some of the MQ concepts outside
of the docs.