stellargraph/stellargraph

View on GitHub
scripts/test_demos.py

Summary

Maintainability
A
0 mins
Test Coverage

Consider possible security implications associated with subprocess module.
Open

import subprocess
Severity: Info
Found in scripts/test_demos.py by bandit

subprocess call with shell=True identified, security issue.
Open

        procout = subprocess.run(
            cmd_line,
            shell=True,
            check=False,
            env=environ,
Severity: Major
Found in scripts/test_demos.py by bandit

Remove this commented out code.
Open

            # stdout=subprocess.PIPE,
Severity: Major
Found in scripts/test_demos.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"

There are no issues that match your filters.

Category
Status