Skip to content

docs: fix performance benchmark script paths - #1536

Open
JJJ-NONAME wants to merge 2 commits into
Agent-Field:devfrom
JJJ-NONAME:fix/1446-performance-readme-path
Open

JJJ-NONAME wants to merge 2 commits into
Agent-Field:devfrom
JJJ-NONAME:fix/1446-performance-readme-path

Conversation

@JJJ-NONAME

Copy link
Copy Markdown

Summary

  • Fix the performance benchmark README command to use the script's actual docs/benchmarks/performance/ path.
  • Keep the script's own example consistent with that path.

Closes #1446.

Verification

  • git diff --check
  • Confirmed both documented paths resolve to the tracked script.
  • make pr-ready was attempted; the repository's POSIX-only syscall tests and shell steps cannot run on this Windows host.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@santoshkumarradha

Copy link
Copy Markdown
Member

Hey, thanks for the PR. Could you sign the CLA when you get a chance so we can review it?

@santoshkumarradha santoshkumarradha added this to the Tests & tooling milestone Sep 27, 2026
@santoshkumarradha santoshkumarradha added hygiene Tests, laws, dead code, duplication — no person-facing change area:tests The suite itself — flakes, harnesses, laws, CI reds labels Sep 27, 2026
@santoshkumarradha

Copy link
Copy Markdown
Member

The two proposed path corrections are byte-identical to the incorporated #1498 changes, which also include a regression test. This is duplicate coverage, not a separate source-head merge. Superseded by #1632. The aggregate remains draft pending the remaining batch and final checks; this does not indicate a merge into dev.

@santoshkumarradha

Copy link
Copy Markdown
Member

Reopening: the second consolidation is now restricted to Santosh-authored PRs, so this contributor PR remains outside #1632. The earlier consolidation-related closure no longer applies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:tests The suite itself — flakes, harnesses, laws, CI reds hygiene Tests, laws, dead code, duplication — no person-facing change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The performance README points to a measure script at the wrong path

3 participants