Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 5, 2026

  • Review the current release.md documentation structure
  • Add a new section explaining staging branches in Ask Mode
  • Document when and why branchToUpdateSubmodule differs from branchToMirror
  • Explain the concept of staging branches for release validation
  • Verify formatting and build the documentation
  • Run code review and security checks
  • Address PR feedback:
    • Remove benefits bullet list (lines 85-91)
    • Update backport practices to mention "Clean cherry-pick of #<PR_NUMBER>"
    • Move "Existing Release Branches" section earlier
    • Add release/1.8 placeholder and update release notes
    • Update release/1.8 to release/1.8.xxyy with Active Development phase

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Explain the concept of staging branches during Ask Mode and document when/why branchToUpdateSubmodule differs from branchToMirror in the mirror pipeline configuration.

Co-authored-by: mattkur <13772048+mattkur@users.noreply.github.com>
Copilot AI changed the title [WIP] Update release documentation to explain staging branches Document staging branch workflow for Ask Mode releases Feb 5, 2026
Copilot AI requested a review from mattkur February 5, 2026 17:39
Comment on lines 85 to 91
This staged approach provides maintainers with greater control during the
critical Ask Mode phase, allowing them to:

* **Test changes in isolation** before they reach the release branch
* **Batch multiple fixes** together for comprehensive validation
* **Roll back problematic changes** from staging without affecting the release
branch
Copy link
Contributor

Choose a reason for hiding this comment

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

This stuff isn't required.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed in 38e5c4d

Comment on lines +99 to +102
* **`branchToMirror`**: Points to the official release branch (e.g.,
`release/1.7.2511`)
* **`branchToUpdateSubmodule`**: Points to the staging branch (e.g.,
`staging/1.7.2511`)
Copy link
Contributor

Choose a reason for hiding this comment

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

@benhillis , is this right?

Comment on lines 114 to 121
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's move this above ## Marking, Approval Process, Code Flow

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved in 38e5c4d

Comment on lines 61 to 62
Copy link
Contributor

Choose a reason for hiding this comment

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

We want devs to say "Clean cherry pick of #mypr" in the PR description. Update here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in 38e5c4d to include "Clean cherry-pick of #<PR_NUMBER>" guidance

Copy link
Contributor

Choose a reason for hiding this comment

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

Oops, let's also update this line: the next release will require supporting updates from 2411, 2505, and 1.7.2511.
Let's also create a placeholder, called release/1.8., where development happens in main.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in 38e5c4d - added release/1.8 placeholder and updated servicing notes to include 1.7.2511

… move release table, add release/1.8

- Remove the bullet list explaining staging branch benefits (not required)
- Add guidance to include "Clean cherry-pick of #<PR_NUMBER>" for clean cherry-picks
- Move "Existing Release Branches" section above "Marking, Approval Process, Code Flow"
- Add release/1.8 as Stabilization phase
- Update next release notes to include servicing from 1.7.2511

Co-authored-by: mattkur <13772048+mattkur@users.noreply.github.com>
Copilot AI requested a review from mattkur February 5, 2026 18:02
| release/2411 | Out of service | |
| release/2505 | Servicing | Supports runtime servicing from release/2411. |
| release/1.7.2511 | Ask Mode | Supports runtime servicing from release/2411 and release/2505. |
| release/1.8 | Stabilization | Supports runtime servicing from release/2411, release/2505, and release/1.7.2511. |
Copy link
Contributor

Choose a reason for hiding this comment

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

No, call it release/1.8.xxyy and say we are in Active Development

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in d287a16 - changed to release/1.8.xxyy with Active Development phase

Changed release/1.8 to release/1.8.xxyy and updated its phase from Stabilization to Active Development per feedback.

Co-authored-by: mattkur <13772048+mattkur@users.noreply.github.com>
Copilot AI requested a review from mattkur February 5, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants