You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
*Exception Handling can be added for error clarity like subprocess.CalledProcessError
*Add type annotations for better code clarity and type checking.
*Avoid using shell=True for security reasons, especially with dynamic input. Use a list of arguments instead.
Expected behavior
Error handling and security concerns can be dealt with properly
Bug report issues can be followed by a PR without discussion with a core developer in this issue.
The text was updated successfully, but these errors were encountered:
Describe the bug
*Exception Handling can be added for error clarity like subprocess.CalledProcessError
*Add type annotations for better code clarity and type checking.
*Avoid using shell=True for security reasons, especially with dynamic input. Use a list of arguments instead.
Expected behavior
Error handling and security concerns can be dealt with properly
Bug report issues can be followed by a PR without discussion with a core developer in this issue.
The text was updated successfully, but these errors were encountered: