Skip to content

ci: publish with mops 3.5.1, which benchmarks on moc 2 - #30

Merged
Kamirus merged 1 commit into
mainfrom
kamil-claude/mops-publish-3.5.1
Oct 19, 2026
Merged

Kamirus merged 1 commit into
mainfrom
kamil-claude/mops-publish-3.5.1

Conversation

@Kamirus

@Kamirus Kamirus commented Oct 19, 2026

Copy link
Copy Markdown
Collaborator

The first Publish on Mops run for v0.2.2 (run) failed while benchmarking:

Error: benchmarks failed
/home/runner/.cache/mops/moc/2.0.0/moc: unknown option '--legacy-persistence'.

mops publish runs the benchmarks itself. Up to 3.4.2 it hard-coded legacy-persistence flags, and moc 2.0.0 doesn't accept them. The regular mops bench in CI was already fixed, which is why tests passed. mops 3.5.0 makes publish use the same moc flags as bench, so the workflow now pins 3.5.1. With 3.5.1, mops publish --dry-run --no-test on this tree passes locally.

mops publish also gets GITHUB_TOKEN. Since 3.5.0, mops uses it to read release notes, instead of making anonymous requests that a shared runner can find already rate-limited.

Follow-up to #29. After merge, Publish on Mops gets dispatched again with tag v0.2.2.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Kamirus
Kamirus requested a review from a team as a code owner October 19, 2026 12:12
@Kamirus
Kamirus merged commit bea22b5 into main Oct 19, 2026
4 checks passed
@Kamirus
Kamirus deleted the kamil-claude/mops-publish-3.5.1 branch October 19, 2026 12:18
Kamirus added a commit that referenced this pull request Oct 19, 2026
How releases reach Mops, and what they depend on, now lives in
`RELEASING.md` instead of only in PR descriptions. It covers the release
steps, publishing an existing tag again, the `MOPS_IDENTITY_PEM` secret
and the GitHub App the workflows need, and the mops version pin that
broke the first 0.2.2 publish.

The repo had no `AGENTS.md`. The new one follows motoko-core's: setup,
test, benchmark, and format commands, the layout, and a link to
`RELEASING.md`. `CLAUDE.md` imports it. The `Releasing` section of
`README.md` now links to `RELEASING.md` for the details.

Follow-up to
[#29](#29) and
[#30](#30).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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