search

SuperShinyEyes / automation-scripts

Scripts for Automation
GNU General Public License v3.0
3 stars 0 forks source link

Unnecessary subprocess #4

Closed igoyak closed 6 years ago

igoyak commented 6 years ago

https://github.com/YoungxHelsinki/automated-scripts/blob/3b8fd3813c73b96826b5693343e496797a13f9ee/Aalto_computer_status_checker/paniikki_jupyter_launcher.py#L237

You could simply use os.remove(). Or even better, using 'with tempfile.TemporaryDirectory()' the cleanup is automatic.

SuperShinyEyes commented 6 years ago

Agree.