fix: move revenueholdings-license to optional [license] extra with graceful import fallback

- Remove revenueholdings-license>=0.1.0 from required dependencies
- Add license optional-dependencies group with revenueholdings-license
- Make require_license() import conditional: try/except ImportError
- On ImportError, define no-op require_license() and print warning
- Fixes CI breakage caused by revenueholdings-license not on PyPI (COM-79, COM-82, COM-83)

Author: Coding-Dev-Tools

pyproject.toml

@@ -26,7 +26,6 @@ classifiers = [
 dependencies = [
     "click>=8.1.0",
     "rich>=13.0.0",
-    "revenueholdings-license>=0.1.0",
     "pathspec>=0.11.0",
     "pyyaml>=6.0",
 ]
@@ -36,6 +35,7 @@ dev = [
     "pytest>=7.0.0",
     "pytest-cov>=4.0.0",
 ]
+license = ["revenueholdings-license>=0.1.0"]
 
 [project.scripts]
 deadcode = "deadcode.cli:cli"

src/deadcode/cli.py

@@ -10,7 +10,13 @@
 import click
 from rich.console import Console
 from rich.table import Table
-from revenueholdings_license import require_license
+try:
+    from revenueholdings_license import require_license
+except ImportError:
+    import warnings
+    warnings.warn("revenueholdings-license not installed; license checks skipped", stacklevel=2)
+    def require_license(product: str) -> None:  # type: ignore[misc]
+        pass
 
 from . import __version__
 from .config import DeadCodeConfig