Skip to content

Updated detailed diff algo to unordered#57

Open
gerrycampion wants to merge 16 commits into
mainfrom
unordered-diff-results
Open

Updated detailed diff algo to unordered#57
gerrycampion wants to merge 16 commits into
mainfrom
unordered-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-rules-engine#1769

@gerrycampion gerrycampion changed the title Updated detailed diff algo Updated detailed diff algo to unordered Jun 18, 2026
@gerrycampion gerrycampion added this to the v1.0.0 milestone Jun 18, 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.

1 participant