Showing 10 of 10 total issues
Function get_config_from_argv
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring. Open
def get_config_from_argv(argv: list):
"""Search for a config file option in command line"""
for index, arg in enumerate(argv):
# manage "=" sign
if arg.find('=') != -1 and arg.split('=')[0] == '--config':
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function _merge_dictionaries
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def _merge_dictionaries(dict1, dict2):
for key, val in dict1.items():
if isinstance(val, dict):
dict2_node = dict2.setdefault(key, {})
_merge_dictionaries(val, dict2_node)
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function exec_cmd
has 6 arguments (exceeds 4 allowed). Consider refactoring. Open
def exec_cmd(ctx: Context, user: str, container: str, command: tuple, tty: bool, workdir: str):
Function main
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def main():
"""Call the CLI Script."""
try:
for alias, conf in get_aliases().items():
if conf is None:
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function exec_cmd
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
def exec_cmd(self,
Function start
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
def start(ctx: Context, container: str, pull: bool, recreate: bool, proxy: bool):
Function restart
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
def restart(ctx: Context, container: str, pull: bool, recreate: bool, proxy: bool):
Function run_commands
has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring. Open
def run_commands(ctx: Context, extra_args: tuple, tty: bool):
"""Run commands for a specific alias"""
commands = ctx.obj['STAKKR'].get_config()['aliases'][ctx.command.name]['exec']
for command in commands:
user = command['user'] if 'user' in command else 'root'
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Avoid too many return
statements within this function. Open
return False, "Couldn't clone {} ({})".format(url, error)
Function _print_errors
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
def _print_errors(result: subprocess.Popen):
"""Print messages sent to the STDERR."""
num = 0
for line in result.stderr:
err = line.decode()
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"