Skip to content

Workflow: Add Missing Defaults lists what it added - #185

Merged
Osayi-ANL merged 1 commit into
mainfrom
change/workflow-add-missing-defaults-report
Oct 21, 2026
Merged

Osayi-ANL merged 1 commit into
mainfrom
change/workflow-add-missing-defaults-report

Conversation

@pecomyint

Copy link
Copy Markdown
Collaborator

Stacked on #184 — review/merge that one first; this diff shows only this PR's change.

Summary

  • seed_defaults() and the profile merge return the paths they add; the dialog shows the list under Show Details instead of a bare count.

Test plan

  • pytest tests/unit passes; ruff check src/dashpva/ clean
  • Updated test_seed_defaults.py; new test_deep_merge_reports_added_profile_paths

🤖 Generated with Claude Code

@Osayi-ANL Osayi-ANL left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@Osayi-ANL
Osayi-ANL force-pushed the change/workflow-profile-controls branch from 862b3a2 to 732361c Compare October 21, 2026 20:50
@Osayi-ANL
Osayi-ANL changed the base branch from change/workflow-profile-controls to main October 21, 2026 20:52
@Osayi-ANL
Osayi-ANL force-pushed the change/workflow-add-missing-defaults-report branch from b67d776 to fd62d35 Compare October 21, 2026 20:52
@Osayi-ANL
Osayi-ANL merged commit 1922f21 into main Oct 21, 2026
3 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