Skip to content

ci: list every nightly report on the Pages root - #6079

Merged
kixelated merged 3 commits into
mainfrom
ci/pages-root
Oct 20, 2026
Merged

kixelated merged 3 commits into
mainfrom
ci/pages-root

Conversation

@kixelated

@kixelated kixelated commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

The nightly reports site (now https://git.moq.dev/, formerly https://moq-dev.github.io/moq/) returns 404 at its root because the nightly Benchmark site job only writes bench/index.html. The landing step now also writes a root index.html. It links every top-level report directory on gh-pages that has an index.html, labeled by that page's <title>. Today that's only bench/. The relay report from moq-dev/moq#6046 (relays/), and any later report, shows up without editing this page, since each writer owns its own directory. The publisher commits the root page with bench/index.html.

Context: Pages was enabled, and the first deploy succeeded today when the Benchmark site job in run 39421743227 was re-run.

bench/README.md now points at git.moq.dev, and notes that re-running the deploy job in place fails.

  • Public API / wire impact: none.
  • Tests: bench-report.test.py checks that the root lists bench/ and a second report directory. just check passes. One earlier just check run had a single Python test error in an unrelated .github/justfile recipe that did not reproduce.

🤖 Generated with Claude Code

(Written by Claude Opus 5.5)

The site only wrote bench/index.html, so https://moq-dev.github.io/moq/
returned 404.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 5 minutes.

Check out review usage here.

View limit details

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

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6cec2ce3-9839-4868-a462-e3337b321045
📥 Commits

Reviewing files that changed from the base of the PR and between 9a2663e and e7f3077.

📒 Files selected for processing: 4
  • bench/README.md
  • sh/gh/bench-publish.sh
  • sh/gh/bench-report.py
  • sh/gh/bench-report.test.py
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-20T18:30:41.839020Z 1b9dacb New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Each gh-pages writer owns one directory, so the root links whichever reports
exist, labeled by their page titles, instead of forwarding only to bench/.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated kixelated changed the title ci: redirect the nightly Pages root to the benchmark charts ci: list every nightly report on the Pages root Oct 20, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated

Copy link
Copy Markdown
Collaborator Author

Merge summary for e7f3077:

  • The nightly Pages root (https://git.moq.dev/) now lists every top-level gh-pages report directory that has an index.html, labeled by its <title>. Later reports such as relays/ show up without edits.
  • bench/README.md points at git.moq.dev and documents that re-running the deploy job in place fails.
  • Review: Codex found nothing on 1b9dacb. The only change since is the docs-only README edit in e7f3077.
  • Local checks: just check passes, and sh/gh/bench-report.test.py passes. The first just check run hit a 15s timeout in the unrelated bench/relay.test.py. It passed 3/3 runs on its own and on the full re-run, so it is a load flake.
  • Public API / wire impact: none.

Enabling auto-merge.

(Written by Claude Opus 5.5)

@kixelated
kixelated added this pull request to the merge queue Oct 20, 2026
Merged via the queue into main with commit fa61ac0 Oct 20, 2026
5 checks passed
@kixelated
kixelated deleted the ci/pages-root branch October 20, 2026 20:15
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.

1 participant