Skip to content

feat(validation): link run and fixture in regression summary - #470

Merged
hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/regression-summary-result-links
Oct 10, 2026
Merged

hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/regression-summary-result-links

Conversation

@hdamker

@hdamker hdamker commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

enhancement/feature

What this PR does / why we need it:

The regression runner's markdown summary now has Run and Fixture columns per branch. Run links to the camara-validation.yml Actions run whose findings were compared; Fixture links to the branch's .regression/regression-expected.yaml. Both fall back to - when the repo or run ID is not available. The runner is shared by the ReleaseTest canary sweep and the CommonalitiesTest sweep, so both summaries get the links.

  • A reader can jump from a failing row straight to the run logs and the expected findings.
  • DiffReport gains optional repo and run_id, so existing callers are unaffected.
  • The example table in validation/docs/regression-testing.md is updated.

Which issue(s) this PR fixes:

N/A

Special notes for reviewers:

None.

Changelog input

 release-note

Additional documentation

docs

@hdamker
hdamker requested review from Kevsy and rartych as code owners October 10, 2026 05:45

@rartych rartych 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

@hdamker
hdamker merged commit 5078e37 into camaraproject:main Oct 10, 2026
8 checks passed
@hdamker
hdamker deleted the fix/regression-summary-result-links branch October 10, 2026 06:49
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