Skip to content
This repository was archived by the owner on Sep 17, 2026. It is now read-only.

Add status tags to ESR rows - #9159

Closed
amyblais wants to merge 1 commit into
masterfrom
claude/pr-9158-solution-check-ae0lkz
Closed

amyblais wants to merge 1 commit into
masterfrom
claude/pr-9158-solution-check-ae0lkz

Conversation

@amyblais

Copy link
Copy Markdown
Member

No description provided.

The two Extended Support Release rows in the release policy timeline
carried only a `crit` tag and their dates, with no `done`/`active`
status. Consumers that read the chart to determine which releases are
still supported treat an untagged row as supported, which is why the
cherry-pick automation kept targeting release-10.11 after that ESR
reached end of life on 2026-08-15.

Tag both ESR rows:
- v10.11 & Desktop App v5.13 -> `crit, done` (support ended 2026-08-15)
- v11.7 & Desktop App v6.2 -> `crit, active` (supported to 2027-05-15)

Note the tags are bare keywords. In Mermaid only the first colon
separates the task name from its data, so a second colon (`:crit,
:done,`) fails the tag regex and the token is consumed as the task id
instead, leaving the row untagged and unchanged in the rendered chart.
Both rows keep `crit` so they still render as red ESR bars per the
timeline legend.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vb9NiCSQmQrk1syZGQMwhT
Copilot AI lite review requested due to automatic review settings August 21, 2026 07:58
@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 12963b5e-82ac-483d-9b82-8c0e8eb54162

📥 Commits

Reviewing files that changed from the base of the PR and between 8b116ca and 8dbd03b.

📒 Files selected for processing (1)
  • source/product-overview/release-policy.md

Limit details: You’ve used all 2 included reviews currently available.


📝 Walkthrough

Walkthrough

The release policy timeline updates Mermaid status formatting for the v10.11 and v11.7 ESR entries.

Changes

Release timeline updates

Layer / File(s) Summary
ESR timeline status formatting
source/product-overview/release-policy.md
The v10.11 entry removes spacing before the status separator. The v11.7 entry adds the active status while retaining ESR styling and support dates.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 8dbd0

This is a small, localized documentation change with no actionable merge-blocking risk remaining after normal checks and review.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided, so its relevance to the changeset cannot be confirmed. Add a brief description that explains the Mermaid styling changes for the v10.11 and v11.7 ESR rows.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the change to ESR row status tags.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/pr-9158-solution-check-ae0lkz

Usage-based review receipt

Note

This review was completed with usage-based billing: files reviewed beyond your plan's included limits are billed at $0.25/file. Track spend and usage in your billing settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI 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.

Pull request overview

Updates the Mattermost release policy Mermaid Gantt chart to explicitly label ESR (Extended Support Release) rows with status tags so the chart renders ESR lifecycle states consistently with other release rows.

Changes:

  • Marked the v10.11 ESR row as done to reflect its end-of-support date.
  • Marked the v11.7 ESR row as active to reflect its ongoing support window.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

Copy link
Copy Markdown
Contributor

Newest code from mattermost has been published to preview environment for Git SHA 8dbd03b

@amyblais amyblais closed this Aug 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants