Update logger initialization to use module-specific loggers #140

Changed "logger = logging.getLogger()" line of code to "logger = logging.getLogger(__name__)" in all files

Removed the TODO Comments - "# TODO: At least add __name__ as the name for the logger" in all files
#140

src/tirith/cli.py

@@ -16,8 +16,8 @@
 
 from .core import start_policy_evaluation
 
-# TODO: Use at least __name__ for the logger name
-logger = logging.getLogger()
+
+logger = logging.getLogger(__name__)
 
 
 def eprint(*args, **kwargs):

src/tirith/core/core.py

@@ -12,8 +12,8 @@
 from .evaluators import EVALUATORS_DICT
 
 
-# TODO: Use __name__ for the logger name instead of using the root logger
-logger = logging.getLogger()
+
+logger = logging.getLogger(__name__)
 
 
 def get_evaluator_inputs_from_provider_inputs(provider_inputs, provider_module, input_data):

src/tirith/core/evaluators/contained_in.py

@@ -3,8 +3,7 @@
 from .base_evaluator import BaseEvaluator
 from tirith.utils import sort_collections
 
-# TODO: At least add __name__ as the name for the logger
-logger = logging.getLogger()
+logger = logging.getLogger(__name__)
 
 # Checks if :attr: `evaluator_input` is contained in :attr:`evaluator_data`.
 

src/tirith/core/evaluators/contains.py

@@ -3,8 +3,8 @@
 from .base_evaluator import BaseEvaluator
 from tirith.utils import sort_collections
 
-# TODO: At least add __name__ as the name for the logger
-logger = logging.getLogger()
+
+logger = logging.getLogger(__name__)
 
 # Checks if :attr: `evaluator_input` is contained in :attr:`evaluator_data`.
 

src/tirith/core/evaluators/equals.py

@@ -3,8 +3,7 @@
 from .base_evaluator import BaseEvaluator
 from tirith.utils import sort_collections
 
-# TODO: Use __name__ for the logger name instead of using the root logger
-logger = logging.getLogger()
+logger = logging.getLogger(__name__)
 
 # Checks if :attr:`value` is equal to :attr:`other`. Automatically casts values to the same type if possible.
 

src/tirith/core/evaluators/not_contained_in.py

@@ -3,8 +3,8 @@
 from .base_evaluator import BaseEvaluator
 from tirith.utils import sort_collections
 
-# TODO: At least add __name__ as the name for the logger
-logger = logging.getLogger()
+
+logger = logging.getLogger(__name__)
 
 # Checks if :attr: `evaluator_input` is not contained in :attr:`evaluator_data`.
 

src/tirith/core/evaluators/not_contains.py

@@ -3,8 +3,8 @@
 from .base_evaluator import BaseEvaluator
 from tirith.utils import sort_collections
 
-# TODO: At least add __name__ as the name for the logger
-logger = logging.getLogger()
+
+logger = logging.getLogger(__name__)
 
 # Checks if :attr: `evaluator_input` does not contain :attr:`evaluator_data`.
 

src/tirith/core/evaluators/not_equals.py

@@ -3,8 +3,7 @@
 from .base_evaluator import BaseEvaluator
 from tirith.utils import sort_collections
 
-# TODO: Use __name__ for the logger name instead of using the root logger
-logger = logging.getLogger()
+logger = logging.getLogger(__name__)
 
 # Checks if :attr:`value` is not equal to :attr:`other`. Automatically casts values to the same type if possible.
 

src/tirith/providers/infracost/handler.py

@@ -1,7 +1,6 @@
 import logging
 
-# TODO: Add at least __name__ as the name of the logger
-logger = logging.getLogger()
+logger = logging.getLogger(__name__)
 
 
 def __get_all_costs(operation_type, input_data):

src/tirith/providers/sg_workflow/handler.py

@@ -1,7 +1,7 @@
 import logging
 
-# TODO: Add at least __name__ as the name of the logger
-logger = logging.getLogger()
+
+logger = logging.getLogger(__name__)
 
 
 def __getValue(key, data):

src/tirith/providers/terraform_plan/handler.py

@@ -43,7 +43,7 @@ def _get_exp_attribute(split_expressions, input_data):
                     final_data.append(val)
     return final_data
 
-
+    
 def provide(provider_inputs, input_data):
     # """Provides the value of the attribute from the input_data"""
     outputs = []

src/tirith/utils.py

@@ -1,6 +1,6 @@
 import logging
 
-logger = logging.getLogger()
+logger = logging.getLogger(__name__)
 
 
 def sort_collections(inputs):