Skip to content

Use PR head commits for conformance artifacts - #137

Merged
lucarlig merged 1 commit into
mainfrom
user/luca/conformance-pr-head
Sep 7, 2026
Merged

Use PR head commits for conformance artifacts#137
lucarlig merged 1 commit into
mainfrom
user/luca/conformance-pr-head

Conversation

@lucarlig

@lucarlig lucarlig commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

/conformance can time out waiting for an artifact when main advances and the PR's synthetic merge commit changes. Build and name the CI artifact using the PR head SHA, then check out that same SHA for conformance. Increase artifact retention from one day to 90 days so later conformance runs can reuse the PR build.

Update the testing wiki to describe the head-commit contract. Existing PRs need a fresh CI build using the updated workflow; expired artifacts are not restored.

Validation: workflow actionlint, cargo fmt, Clippy with -D warnings, cargo deny, workspace build, and wiki build/tests passed. Workspace nextest: 247 passed, 11 skipped. The GitHub-hosted conformance job has not been rerun.

Signed-off-by: lucarlig <luca.carlig@ibm.com>
@lucarlig
lucarlig marked this pull request as ready for review September 7, 2026 12:49

@cafalchio cafalchio 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.

LGTM

@lucarlig
lucarlig merged commit 9df7fe9 into main Sep 7, 2026
11 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