Skip to content

feat: add --require-license strict mode flag#36

Merged
Coding-Dev-Tools merged 4 commits into
mainfrom
improve/configdrift-require-license-20260629
Jun 29, 2026
Merged

feat: add --require-license strict mode flag#36
Coding-Dev-Tools merged 4 commits into
mainfrom
improve/configdrift-require-license-20260629

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

feat: add --require-license strict mode flag

Closes #32

Changes

  • Added --require-license flag to the main CLI group
  • Supported REVENUEHOLDINGS_REQUIRE_LICENSE=1 environment variable
  • When set and revenueholdings-license is not installed: exits with a clear error message
  • When set and license check fails: propagates the error

Usage

# Exit with error if license not installed
configdrift --require-license <command>

# Or via env var
REVENUEHOLDINGS_REQUIRE_LICENSE=1 configdrift <command>

Acceptance Criteria

  • --require-license flag added to main CLI group
  • REVENUEHOLDINGS_REQUIRE_LICENSE env var supported
  • When set and package not installed: exit with clear error
  • When set and license check fails: propagate the error
  • Documented in README (follow-up)

Generated by Claude Code

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown

🤖 Automated Code Review

✅ Ruff Lint — No issues

⚠️ Ruff Format — Formatting needed

Would reformat: src/configdrift/cli.py
Would reformat: src/configdrift/diff.py
Would reformat: src/configdrift/loader.py
Would reformat: tests/test_cli.py
Would reformat: tests/test_coverage_gaps.py
Would reformat: tests/test_diff.py
Would reformat: tests/test_loader.py
7 files would be reformatted, 2 files already formatted

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 1 file(s) changed

 src/configdrift/cli.py | 63 +++++++++++++++++++++++++++++++++++++-------------
 1 file changed, 47 insertions(+), 16 deletions(-)

Verdict: ⚠️ Warnings — Lint/format issues found. Recommend fixing before merge.

Automated by Coding-Dev-Tools/.github reusable workflow.

@Coding-Dev-Tools Coding-Dev-Tools merged commit 0b3304d into main Jun 29, 2026
5 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.

feat: add --require-license strict mode flag

1 participant