Skip to content

support detailed markdown diff#1769

Merged
SFJohnson24 merged 7 commits into
mainfrom
update-diff-results
Jun 23, 2026
Merged

support detailed markdown diff#1769
SFJohnson24 merged 7 commits into
mainfrom
update-diff-results

Conversation

@gerrycampion

@gerrycampion gerrycampion commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

For Published rules testing, this improves the reporting of failed tests in the detailed report.

  • Uses markdown table to display differences
  • Aligns differences using closest row match instead of strictly row number
  • Uploads both the expected and actual results to artifacts instead of only the expected results

Action run before update: https://github.com/cdisc-org/cdisc-rules-engine/actions/runs/27967250482
Action run after update: https://github.com/cdisc-org/cdisc-rules-engine/actions/runs/27973685763

Requires PR: cdisc-org/cdisc-open-rules#57

@SFJohnson24 SFJohnson24 merged commit 66dc615 into main Jun 23, 2026
4 of 6 checks passed
@SFJohnson24 SFJohnson24 deleted the update-diff-results branch June 23, 2026 18:14
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