Skip to content

Add CADT v2 config format for version >= 1.7.26#467

Merged
TheLastCicada merged 4 commits intomainfrom
cadt-v2-config
Feb 28, 2026
Merged

Add CADT v2 config format for version >= 1.7.26#467
TheLastCicada merged 4 commits intomainfrom
cadt-v2-config

Conversation

@TheLastCicada
Copy link
Contributor

CADT 1.7.26 introduces a new config structure with separate V1/V2 sections, each with their own governance, DB, and API key settings. The role now detects the version and selects the appropriate config template, with V1/V2 defaults inheriting from existing variables for backward compatibility.

CADT 1.7.26 introduces a new config structure with separate V1/V2
sections, each with their own governance, DB, and API key settings.
The role now detects the version and selects the appropriate config
template, with V1/V2 defaults inheriting from existing variables for
backward compatibility.
CADT 1.7.26 reads config.yaml from the cadt/ directory directly
instead of cadt/v1/. The old path is preserved for older versions.
cmmarslender
cmmarslender previously approved these changes Feb 28, 2026
@cmmarslender cmmarslender self-requested a review February 28, 2026 06:28
V2 should not inherit from V1's legacy single-value variables.
Defaulting to null/false ensures V2 must be explicitly configured
when enabled.
cmmarslender
cmmarslender previously approved these changes Feb 28, 2026
Line 46 had 9 spaces (odd), editorconfig requires multiples of 2.
@TheLastCicada TheLastCicada merged commit 0727413 into main Feb 28, 2026
11 checks passed
@TheLastCicada TheLastCicada deleted the cadt-v2-config branch February 28, 2026 16:20
TheLastCicada added a commit that referenced this pull request Mar 2, 2026
Resolve conflicts in cadt/tasks/main.yml: keep main's conservative
version check logic and dynamic config path from PR #467.
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