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
task_analyse_pylint executes Pylint as a subprocess and checks stderr for errors. However, if Pylint fails in an unexpected way (e.g. the task is killed) the stderr may be empty, thus causing the script to incorrectly assume Pylint completed successfully.
We should check the process exit status as an additional safegaurd against the possibility of missing warnings (a threat to the internal validity of the study).
The text was updated successfully, but these errors were encountered:
anjsimmo
changed the title
Exit status of Pylint subprocesses not checked
Exit status of Pylint subprocess not checked
Apr 6, 2020
Additional details: In some cases, Pylint is printing warnings to stderr that can be safely ignored. It would be much better to check the return code of Pylint rather than attempting to use stderr.
task_analyse_pylint
executesPylint
as a subprocess and checks stderr for errors. However, if Pylint fails in an unexpected way (e.g. the task is killed) the stderr may be empty, thus causing the script to incorrectly assumePylint
completed successfully.We should check the process exit status as an additional safegaurd against the possibility of missing warnings (a threat to the internal validity of the study).
The text was updated successfully, but these errors were encountered: