Skip to content

Updates - #7

Open
biow0lf wants to merge 5 commits into
mainfrom
updates-7
Open

Updates#7
biow0lf wants to merge 5 commits into
mainfrom
updates-7

Conversation

@biow0lf

@biow0lf biow0lf commented Sep 11, 2026

Copy link
Copy Markdown
Member

No description provided.

@biow0lf
biow0lf requested a balanced review from Copilot September 11, 2026 18:29
@biow0lf biow0lf self-assigned this Sep 11, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The release notes contain malformed HTML and are not linked from the site.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds v0.11.0 release documentation and marks the site as outdated.

Changes:

  • Documents v0.11.0 changes and fixes.
  • Adds an old-documentation warning.
File summaries
File Description
source/release_notes/v0.11.0.html.erb Adds v0.11.0 release notes.
source/index.html.erb Adds a warning banner.
Review details

Suppressed comments (1)

source/release_notes/v0.11.0.html.erb:108

  • The second <code> element is closed with a Markdown backtick, and the third path also uses literal backticks. This leaves invalid HTML and can cause code styling to leak into later content. Close each path with matching HTML tags.
  <li class="list-group-item">Remove dead assets for HipChat: <code>app/assets/images/hipchat_create.png</code>, <code>app/assets/images/hipchat_goto.png` and `app/assets/images/hipchat_inactive.png`.</li>
  • Files reviewed: 2/2 changed files
  • Comments generated: 2
  • Review effort level: Balanced

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

Comment thread source/release_notes/v0.11.0.html.erb Outdated
Comment thread source/release_notes/v0.11.0.html.erb

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The release wording is inaccurate, the notes are not indexed, and the warning lacks a clear destination.

Review details

Suppressed comments (3)

Previously missed (2) — in code that hasn't changed since the last review.

source/index.html.erb:6

  • The warning says the old documentation is “here,” but provides neither a link nor a concrete location, so readers cannot tell whether it refers to this page or another site. Link “here” to the legacy documentation, or state explicitly that the current site is the legacy documentation.
    source/release_notes/v0.11.0.html.erb:13
  • This statement is already false for v0.11.0 itself: v0.11.1 was released afterward and explicitly adds MongoDB 8.3 support. The upstream v0.11.0 notes describe v0.11 as the last MongoDB-supporting branch, not v0.11.0 as the final release, so retain that release-series meaning.

source/release_notes/v0.11.0.html.erb:9

  • This new release-note content is not discoverable from the release-notes landing page: source/release_notes/index.html.erb still contains only an empty “About” link and no link to v0.11.0. Add this release to that index so users can navigate to the page without knowing its URL.
<h2>Notable changes:</h2>
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

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