CiscoUcs/ucsmsdk

View on GitHub
ucsmsdk/ucsexception.py

Summary

Maintainability
A
0 mins
Test Coverage

Remove this commented out code.
Open

    # message = 'Authentication failed'
Severity: Major
Found in ucsmsdk/ucsexception.py by sonar-python

Programmers should not comment out code as it bloats programs and reduces readability.

Unused code should be deleted and can be retrieved from source control history if required.

See

  • MISRA C:2004, 2.4 - Sections of code should not be "commented out".
  • MISRA C++:2008, 2-7-2 - Sections of code shall not be "commented out" using C-style comments.
  • MISRA C++:2008, 2-7-3 - Sections of code should not be "commented out" using C++ comments.
  • MISRA C:2012, Dir. 4.4 - Sections of code should not be "commented out"

Rename function "UcsWarning" to match the regular expression ^[a-z_][a-z0-9_]{2,}$.
Open

def UcsWarning(warn_str):
Severity: Major
Found in ucsmsdk/ucsexception.py by sonar-python

Shared coding conventions allow teams to collaborate efficiently. This rule checks that all function names match a provided regular expression.

Noncompliant Code Example

With the default provided regular expression: ^[a-z_][a-z0-9_]{2,30}$

def MyFunction(a,b):
    ...

Compliant Solution

def my_function(a,b):
    ...

There are no issues that match your filters.

Category
Status