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
As mentioned in #546, we should start making the code more robust by replacing os.system calls like the one in tarfiles.tar_up() with either call or run from the subprocess module.
So for this particular case:
Make the appropriate os.system -> subprocess.run/call substitution in tarfiles.tar_up()
There are a few notes in tarfiles.py as to what needs to change with the error handling after the previous item is done. Make those changes.
Add to unit tests to make sure this method is still acting correctly
Change integration tests in test_full_home.py to observe new error handling
The text was updated successfully, but these errors were encountered:
As mentioned in #546, we should start making the code more robust by replacing
os.system
calls like the one intarfiles.tar_up()
with eithercall
orrun
from thesubprocess
module.So for this particular case:
os.system
->subprocess.run/call
substitution intarfiles.tar_up()
tarfiles.py
as to what needs to change with the error handling after the previous item is done. Make those changes.test_full_home.py
to observe new error handlingThe text was updated successfully, but these errors were encountered: