vpoulailleau/simplelogging

View on GitHub
simplelogging/__init__.py

Summary

Maintainability
A
2 hrs
Test Coverage

Function get_logger has 8 arguments (exceeds 4 allowed). Consider refactoring.
Open

def get_logger(
Severity: Major
Found in simplelogging/__init__.py - About 1 hr to fix

    Function configure_main_logger has 8 arguments (exceeds 4 allowed). Consider refactoring.
    Open

    def configure_main_logger(
    Severity: Major
    Found in simplelogging/__init__.py - About 1 hr to fix

      Function "configure_main_logger" has 8 parameters, which is greater than the 7 authorized.
      Open

          logger,
          logger_level=DEBUG,
          console=True,
          console_format=DEFAULT_CONSOLE_FORMAT,
          console_level=INFO,
      Severity: Major
      Found in simplelogging/__init__.py by sonar-python

      A long parameter list can indicate that a new structure should be created to wrap the numerous parameters or that the function is doing too many things.

      Noncompliant Code Example

      With a maximum number of 4 parameters:

      def do_something(param1, param2, param3, param4, param5):
          ...
      

      Compliant Solution

      def do_something(param1, param2, param3, param4):
          ...
      

      Function "get_logger" has 8 parameters, which is greater than the 7 authorized.
      Open

          name=None,
          logger_level=DEBUG,
          console=True,
          console_format=DEFAULT_CONSOLE_FORMAT,
          console_level=INFO,
      Severity: Major
      Found in simplelogging/__init__.py by sonar-python

      A long parameter list can indicate that a new structure should be created to wrap the numerous parameters or that the function is doing too many things.

      Noncompliant Code Example

      With a maximum number of 4 parameters:

      def do_something(param1, param2, param3, param4, param5):
          ...
      

      Compliant Solution

      def do_something(param1, param2, param3, param4):
          ...
      

      There are no issues that match your filters.

      Category
      Status