Skip to content

Working PR for ADR-191: Refactor DVC pipeline into proper OOP design patterns#243

Draft
jodavis wants to merge 1 commit into
mainfrom
feature/ADR-191-ml-pipeline
Draft

Working PR for ADR-191: Refactor DVC pipeline into proper OOP design patterns#243
jodavis wants to merge 1 commit into
mainfrom
feature/ADR-191-ml-pipeline

Conversation

@jodavis

@jodavis jodavis commented Jul 24, 2026

Copy link
Copy Markdown
Owner

ADR-191: Refactor DVC pipeline into proper OOP design patterns

Change record:

  • Spec added

Adds the finalized design spec and 21-task breakdown for refactoring the
DVC pipeline into proper OOP design patterns, superseding the discarded
earlier implementation attempt and its task breakdown.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@jodavis
jodavis marked this pull request as draft July 24, 2026 23:33
@github-actions

Copy link
Copy Markdown

Test Results

401 tests  ±0   401 ✅ ±0   1m 59s ⏱️ -6s
  5 suites ±0     0 💤 ±0 
  5 files   ±0     0 ❌ ±0 

Results for commit c4261cd. ± Comparison against base commit 8f1b8b7.

This pull request removes 3 and adds 2 tests. Note that renamed tests count towards both.
,False)
AdaptiveRemote.Services.ProgrammaticSettings.PersistSettingsTests ‑ PersistSettings_Set_ValidatesKeyNameAsync (Hello
AdaptiveRemote.Services.ProgrammaticSettings.PersistSettingsTests ‑ PersistSettings_Set_ValidatesValueAsync (Invalid
AdaptiveRemote.Services.ProgrammaticSettings.PersistSettingsTests ‑ PersistSettings_Set_ValidatesKeyNameAsync (Hello
,False)
AdaptiveRemote.Services.ProgrammaticSettings.PersistSettingsTests ‑ PersistSettings_Set_ValidatesValueAsync (Invalid
,False)

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