Skip to content

Fix research page duplicate h1 and stale WIP blog badge - #141

Merged
VatsalSy merged 1 commit into
mainfrom
rayleigh-cloud/research-content-fixes-84f6
Oct 12, 2026
Merged

VatsalSy merged 1 commit into
mainfrom
rayleigh-cloud/research-content-fixes-84f6

Conversation

@VatsalSy

@VatsalSy VatsalSy commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Remove the markdown # Research heading from _research/index.md so the page keeps a single h1 from h1.research-page__title in the research layout.
  • Drop the first WIP entry’s stale “Blog | Coming Soon” badge (linked to a 404 ToDo URL). No Worthington-jets post exists yet on blogs.comphy-lab.org, so the badge is removed rather than rewritten.

Verification notes

  • TOC JS (page-research.js) builds from .research-content h2 only; paper anchors wrap h3 ids — both unaffected.
  • Confirmed no matching blog post via blogs.comphy-lab.org sitemap/content index.
  • Content-only change; no CSS touched.
  • Built /research/ has 1 h1 (h1.research-page__title).

Test plan

  • ./scripts/lint-check.sh
  • npm test (28 suites / 210 tests)
  • ./scripts/build.sh
  • Confirm built /research/ has exactly one h1
Open in Web Open in Cursor 

Summary by CodeRabbit

  • Documentation
    • Removed the standalone “Research” heading from the research page.
    • Removed the Blog badge and link from the “Self-similar Worthington jets” entry; its arXiv and GitHub links remain.

Remove the markdown # Research heading that duplicated the layout
title, and drop the Worthington jets Blog Coming Soon badge that
pointed at a 404 ToDo URL (no matching post on blogs.comphy-lab.org).

Co-authored-by: Vatsal Sanjay | CoMPhy Lab <vatsal.sanjay@comphy-lab.org>
@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 60a5bf3b-3ba8-4028-b443-6e83eee218af

📥 Commits

Reviewing files that changed from the base of the PR and between 2711e98 and 1d9bc8d.


📒 Files selected for processing (1)
  • _research/index.md

💤 Files with no reviewable changes (1)
  • _research/index.md

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📜 Recent review details
⏰ Context from checks skipped due to timeout. (1)
  • GitHub Check: copilot-pull-request-reviewer


📝 Walkthrough
📝 Walkthrough

Walkthrough

The research index no longer has a standalone “Research” heading. The “Self-similar Worthington jets” entry no longer includes its Blog badge and link; its arXiv and GitHub links remain.

Changes

Research index

Layer / File(s) Summary
Research index content
_research/index.md
The standalone “Research” heading and the Blog badge and link for “Self-similar Worthington jets” were removed. The entry’s arXiv and GitHub links remain.

Priority: ⬇️ Low

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

Change: Bug fix


Merge Risk: ⚪ Minimal · up to 1d9bc

No actionable issue is identified; the research-page cleanup is mergeable after normal checks.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
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…
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.
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly summarises both main changes: removing the duplicate research page heading and removing the stale WIP blog badge.

✨ Finishing Touches
✨ Simplify code
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@VatsalSy
VatsalSy marked this pull request as ready for review October 12, 2026 09:46
Copilot AI balanced review requested due to automatic review settings October 12, 2026 09:46
@VatsalSy

Copy link
Copy Markdown
Member Author

@coderabbitai review

@comphy-bot

comphy-bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

comphy-bot review status for comphy-lab/comphy-lab.github.io#141 at 1d9bc8d1: completed

Review completed for this exact pull request head.

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor
⚠️ Action not completed

Already reviewed the last commit. Use @coderabbitai full review to rerun a review of the entire changeset.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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.

🟢 Approval recommended

The focused content changes are correct and preserve existing research-page behavior.

0 open findings

What changed in this PR

Removes redundant and stale research-page content.

Changes:

  • Removes the duplicate Markdown h1.
  • Removes a broken “Blog | Coming Soon” badge.
File Description
_research/​index.md Removes duplicate heading and stale blog link.

🧠 Review effort: Balanced


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

@comphy-bot
comphy-bot self-requested a review October 12, 2026 09:47

@comphy-bot comphy-bot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Review

Reviewed the complete content-only diff in _research/index.md (+0/-3). Removing the standalone Markdown heading and the stale Blog badge preserves the front matter, topic headings, arXiv/GitHub links, and research image.

No actionable findings or P0/P1 blockers identified.

Verification: exact-head build, content-rules, and PR-hygiene checks passed; deterministic patch checks passed. Peer reviews and threads were considered. No PR code was executed and no local build/test rerun was performed.

@VatsalSy
VatsalSy merged commit 84b536d into main Oct 12, 2026
6 checks passed
@VatsalSy
VatsalSy deleted the rayleigh-cloud/research-content-fixes-84f6 branch October 12, 2026 10:14
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.

3 participants