Skip to content

Removing Arena Bench Score Files - #2886

Closed
macanderson wants to merge 2 commits into
mainfrom
release/0.9.0
Closed

Removing Arena Bench Score Files#2886
macanderson wants to merge 2 commits into
mainfrom
release/0.9.0

Conversation

@macanderson

Copy link
Copy Markdown
Owner

DELETE ARENABENCH RESULT FILES NOW GITIGNORED

…s-page section

The `release.minor` prefix is the part that does the work: auto-tag.yml
computes the next version from the newest tag (v0.8.44), not from Cargo.toml,
and reads the bump marker from the subjects of every commit since that tag. So
the stamp below records the intent and this subject is what makes the next
release v0.9.0 — after which the ordinary per-merge patch bumps run 0.9.1,
0.9.2, … off the corrected minor.

- scripts/sync-versions.sh stamps 0.9.0: [workspace.package].version, the
  workspace-member entries in Cargo.lock, and the build-from-source Homebrew
  formula. Running the canonical script rather than hand-editing four places
  keeps this byte-identical in effect to the release commit auto-tag will cut.

- CHANGELOG.md gains the [0.9.0] section covering the whole 0.8 line (123
  non-release commits, v0.8.0..HEAD). This is the "release PR that deliberately
  writes its own section" exception in CHANGELOG.md's contributor note:
  changelog-roll.sh is idempotent and leaves a version that already has a
  section alone, so the roll at release time will not duplicate or overwrite it.
  That section IS the website release page — /releases is a static server
  component over getReleases(), which parses this file at build time, so there
  is no second copy under website/ to keep in sync.

- website/src/lib/changelog.0-9-0.test.ts is the witness, following the
  changelog.0-8-0.test.ts pattern, plus one assertion that file cannot carry:
  that [0.9.0] is releases[0]. /releases takes the first parsed section as the
  current line, so a hand-written section appended at the bottom parses cleanly
  and still renders the previous line as current.

Witness: the two new tests fail on HEAD (`git show HEAD:CHANGELOG.md` contains
no `## [0.9.0]` heading) and pass here — `npm test` in website/, 18 passing.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry, we are unable to review this pull request

The GitHub API does not allow us to fetch diffs exceeding 300 files, and this pull request has 2983

@macanderson
macanderson marked this pull request as ready for review August 11, 2026 22:41

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry, we are unable to review this pull request

The GitHub API does not allow us to fetch diffs exceeding 20000 lines

@vercel vercel Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Additional Suggestion:

The PR meant to delete and gitignore ~2,967 cloud run artifact files, but instead committed them with no ignore rule, tracking ephemeral scratch (SQLite WAL/SHM DBs, .lock files, session backups, trajectories).

Fix on Vercel

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