Repository navigation
feat(lineage): add conditional validation summary chip - #1540
Conversation
DRC-4261 Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Jared Scott <jared.scott@datarecce.io>
DRC-4261 Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Jared Scott <jared.scott@datarecce.io>
DRC-4261 Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Jared Scott <jared.scott@datarecce.io>
b5e58fb to
340da8d
Compare
Code Review: PR #1540SHA BlockersNone. Issues
Notes
|
wcchang1115
left a comment
There was a problem hiding this comment.
Reviewed at 340da8d. NO-GO — 1 issue, 3 notes: #1540 (comment)
Letting the change label and runsAggregatedTag show together put both behind one selectMode check. A node with no action tag then lost its change-category label while action results were on screen, and the graph stopped naming which other models changed. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: Wei-Chun, Chang <wcchang@infuseai.io>
Code Review: PR #1540SHA BlockersNone. IssuesNone. Notes
|
wcchang1115
left a comment
There was a problem hiding this comment.
GO at 707e4f48. Full review: #1540 (comment)
PR checklist
What type of PR is this?
Feature
What this PR does / why we need it:
Which issue(s) this PR fixes:
Closes DRC-4261
Special notes for your reviewer:
feature/drc-4260-aggregate-compact-validation-summaries-by-lineage-nodeDoes this PR introduce a user-facing change?:
Yes.