From 70c3d2f51089016bdbe955179a7ee3467c0f0914 Mon Sep 17 00:00:00 2001 From: fylorn <249551762+fylorn@users.noreply.github.com> Date: Thu, 24 Sep 2026 22:22:29 +0800 Subject: [PATCH] docs(release): CI runs the integration suite on the release PR Co-Authored-By: Claude Opus 5.5 --- docs/operations/release.md | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/docs/operations/release.md b/docs/operations/release.md index 2a9d9f50..9dadf33d 100644 --- a/docs/operations/release.md +++ b/docs/operations/release.md @@ -116,13 +116,15 @@ gh pr create --base main --head release/X.Y.Z \ --body "See CHANGELOG.md [X.Y.Z] for the full notes." ``` -Wait for CI to go green (`Rust Check & Test` + `Frontend Build`). +Wait for CI to go green (`Rust Check & Test` + `Frontend Build`, plus +`Integration Tests`, which runs the full `#[ignore]` suite against +Postgres, Redis and ClickHouse service containers). The PR description is internal — the user-facing release notes live in CHANGELOG.md and are extracted into the GitHub Release body automatically. Don't duplicate them. -CI does not run the `#[ignore]` integration tests. Run them locally -against the release commit before merging (`make test-it`). +`make test-it` locally is optional: the release PR's CI already runs +the same suite on the release commit. ### 6. Squash-merge the PR @@ -299,7 +301,7 @@ make precommit # green make changelog VERSION=X.Y.Z WRITE=1 $EDITOR CHANGELOG.md # review + polish $EDITOR Cargo.toml web/package.json deploy/helm/think-watch/Chart.yaml -make precommit && make test-it # green again +make precommit # green again git commit -am "chore(release): tag X.Y.Z" && git push -u origin release/X.Y.Z gh pr create --base main --head release/X.Y.Z --title "release: vX.Y.Z" gh pr merge --squash --match-head-commit # once CI is green