Rename cli module to _detail.logging

This renaming is effectively two changes, based on code review feedback:

- The module is moved into a separate _detail package, to highlight that
  its contents are an implementation detail and not part of CBI's public
  interface.

- The module is renamed from "cli" to "logging", since the three classes
  it defines (Formatter, MetaWarning, WarningAggregator) are all related
  to logging.

Signed-off-by: John Pennycook <john.pennycook@intel.com>

Author: Pennycook

codebasin/__main__.py

@@ -11,7 +11,7 @@
 import sys
 
 from codebasin import CodeBase, config, finder, report, util
-from codebasin.cli import Formatter, WarningAggregator
+from codebasin._detail.logging import Formatter, WarningAggregator
 
 log = logging.getLogger("codebasin")
 version = "1.2.0"

codebasin/_detail/__init__.py

@@ -0,0 +1,8 @@
+# Copyright (C) 2019-2025 Intel Corporation
+# SPDX-License-Identifier: BSD-3-Clause
+"""
+This package contains implementation details that are not part of the public
+interface of Code Base Investigator. These implementation details are not
+intended to be used by other scripts, and should not be relied upon.
+"""
+import codebasin._detail.logging

codebasin/cli.py renamed to codebasin/_detail/logging.py

@@ -4,7 +4,7 @@
 import logging
 import unittest
 
-from codebasin.cli import Formatter
+from codebasin._detail.logging import Formatter
 
 
 class TestFormatter(unittest.TestCase):

tests/cli/test_formatter.py

@@ -5,7 +5,7 @@
 import re
 import unittest
 
-from codebasin.cli import MetaWarning
+from codebasin._detail.logging import MetaWarning
 
 
 class TestMetaWarning(unittest.TestCase):

tests/cli/test_meta_warning.py

@@ -4,7 +4,7 @@
 import logging
 import unittest
 
-from codebasin.cli import WarningAggregator
+from codebasin._detail.logging import WarningAggregator
 
 
 class TestWarningAggregator(unittest.TestCase):