Skip to content

ci(benchmarks): a published result records specsolve's release version rather than 0.0.1 - #2043

Merged
FBumann merged 3 commits into
mainfrom
claude/specsolve-2032-review-7hrcv0
Oct 14, 2026
Merged

FBumann merged 3 commits into
mainfrom
claude/specsolve-2032-review-7hrcv0

Conversation

@FBumann

@FBumann FBumann commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator

Prompt: Yes, close it and fix the version

Note

The following content was generated by AI.

The measuring job checked out one commit with no tags, so every published result records specsolve as 0.0.1.dev1+g<sha>. fetch-depth: 0 on that checkout gives the version counted from the nearest tag.

Cause, verification, and what is not done

The eight files on main from #2039 all say 0.0.1.dev1+gfbc735a5b; the lock installs 0.9.1.dev31+gfbc735a5b. The version comes from hatch-vcs ([tool.hatch.version] source = "vcs"), which needs the tags and the history back to the nearest one.

Reproduced on main at 61172a8, with setuptools-scm:

checkout version
depth 1, no tags (what actions/checkout does by default) 0.1.dev1+g61172a8af
full history and tags 0.9.1.dev34+g61172a8af

Type ci: the diff touches only .github/, so no changelog line.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NTByC5UJJWvcFj3mvmv83v


Generated by Claude Code

…e refuses a workflow that does not

An apostrophe inside a single-quoted description made the file
unreadable, so GitHub saw no workflow_dispatch trigger and the
trigger:published-bench label's dispatch failed with HTTP 422.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NTByC5UJJWvcFj3mvmv83v
…n rather than 0.0.1

The measuring job checked out one commit with no tags, so the version
counted from no tag. A full checkout gives 0.9.1.dev34+g61172a8af where
a depth-1 one gives 0.1.dev1+g61172a8af.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NTByC5UJJWvcFj3mvmv83v
@FBumann
FBumann enabled auto-merge (squash) October 14, 2026 20:24
@codspeed

codspeed Bot commented Oct 14, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 104 untouched benchmarks
⏩ 307 skipped benchmarks1


Comparing claude/specsolve-2032-review-7hrcv0 (755fc7c) with main (2588883)2

Open in CodSpeed

Footnotes

  1. 307 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

  2. No successful run was found on main (61172a8) during the generation of this report, so 2588883 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

@FBumann
FBumann merged commit 7f30e95 into main Oct 14, 2026
14 checks passed
@FBumann
FBumann deleted the claude/specsolve-2032-review-7hrcv0 branch October 14, 2026 20:28
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