Skip to content

ci(publish): auto-confirm opam publish via OPAMCONFIRMLEVEL=unsafe-yes - #291

Merged
grootstebozewolf merged 1 commit into
mainfrom
claude/busy-mccarthy-vly1n7
Jun 28, 2026
Merged

grootstebozewolf merged 1 commit into
mainfrom
claude/busy-mccarthy-vly1n7

Conversation

@grootstebozewolf

Copy link
Copy Markdown
Owner

Summary

The spatial-algebra-v0.1.2 / robust-predicates-v0.1.1 releases were green and their tarballs attached, but the coq/opam PRs did not open. Logs show opam-publish auto-answered no:

+ yes
+ opam publish --no-browser --repo=coq/opam ...
  - coq-spatial-algebra version 0.1.2 ...
Please confirm the above data. Continue ?  [Y/n] n

opam-publish's confirm checks whether stdin is a tty; in CI it isn't, so the piped yes (and OPAMYES) were ignored and it defaulted to no.

Fix

Use opam's documented non-interactive mechanism, OPAMCONFIRMLEVEL=unsafe-yes, which auto-answers every confirmation without reading input. Drops the yes | pipe and its pipefail handling.

State / honesty

  • This is the correct mechanism per opam docs, but it can't be dry-run here — it'll be verified only by the next release. opam-publish remains best-effort: if it still won't confirm, the run stays green with the tarball attached + the manual command in the job summary.
  • Both existing releases already have their tarballs; only the opam-repo PRs are outstanding. After merge, a fresh tag (*-v0.1.3 / *-v0.1.2) runs the corrected path. (A manual one-liner can also open them immediately from the attached tarballs.)

🤖 Generated with Claude Code


Generated by Claude Code

opam-publish's confirm checks whether stdin is a tty; in CI it is not, so the
previous `yes |` pipe (and OPAMYES) were ignored and it auto-answered "no"
("Please confirm the above data. Continue ? [Y/n] n"), so no coq/opam PR was
opened though the package run was green and the tarball attached.

OPAMCONFIRMLEVEL=unsafe-yes is opam's documented non-interactive mechanism:
it auto-answers every confirmation without reading input. Drop the `yes |`
pipe (and its pipefail dance) in favor of it.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01F8u1xnYFQ8MyQ3mE8z35dU
@grootstebozewolf
grootstebozewolf merged commit 949b9b9 into main Jun 28, 2026
6 checks passed
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