Skip to content

Make release Experiments opt-in - #1424

Merged
JacobStephens2 merged 2 commits into
mainfrom
issue-1423
Oct 27, 2026
Merged

JacobStephens2 merged 2 commits into
mainfrom
issue-1423

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

Summary

-scripts/release.sh [--review] [--no-experiment] <version>
+scripts/release.sh [--review] [--with-experiment] <version>

Skip release Experiments by default. --with-experiment opts in on each invocation, including resumes. Update the release tests, guide and site descriptions.

Closes #1423

Test seams

  • Release script command line: fresh and resumed releases, Experiment opt-in, and interaction with summary review, using the existing release fixture with local Git repositories and fake external commands.

Evidence

  • Before: cargo test --test release experiments::releases_skip_experiments_by_default_and_say_so -- --exact failed with unexpected Experiment call.
    After: the same command passed, exercising default releases with and without --review.
  • /usr/bin/time -p scripts/affected-tests main passed: full Rust suite via the documented cargo test fallback, all 51 site tests, and 3 release-generator tests. Wall time: 1548.50 seconds, including compilation and all selected checks; the focused release target had been compiled earlier.
  • The release suite passed: 85 passed, 2 ignored fixture entry points.
  • cargo fmt --check, bash -n scripts/release.sh, and git diff --check main...HEAD passed.
  • All CI checks passed on 8ee4114163dc18053491e1fbba79da9092db63eb.

Unaddressed findings

  • Standards: None (0 findings).
  • Spec: None (0 findings).
  • Security: None (0 findings).

Security review outcome

No unaddressed introduced findings.

Review coverage

Independent Standards and Spec reviews against main each read all seven changed files, including generated site/docs/index.html. No changed files were left unread; no generated output was excluded.

Merge Danger

Door: two-way

Blast Radius: releases

Release Experiments require explicit opt-in; existing CI and version checks remain.

Built with codex codex-cli 0.164.0 · gpt-6.1-sol · xhigh

@JacobStephens2
JacobStephens2 merged commit 73deb5e into main Oct 27, 2026
18 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-1423 branch October 27, 2026 04:16
This was referenced Oct 27, 2026
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.

make release without experiment default release, so add --with-experiment to release.sh call

1 participant