search

drauger-os-development / edamame

Linux System Installation Utility for Debian-based distributions
GNU General Public License v2.0
8 stars 4 forks source link

minor code clean up and bug fix #71

Closed Batcastle closed 2 years ago

Batcastle commented 2 years ago

Instead of just calling os.mkdir() over and over in try/except blocks, make one call with the exist_ok flag set to True in order to set up the parent directories. Just make sure all os.mkdir() calls are in try/catch blocks.

pep8speaks commented 2 years ago

Hello @Batcastle! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 204:80: E501 line too long (83 > 79 characters) Line 207:80: E501 line too long (82 > 79 characters) Line 218:80: E501 line too long (88 > 79 characters) Line 222:80: E501 line too long (85 > 79 characters) Line 341:80: E501 line too long (80 > 79 characters) Line 344:80: E501 line too long (87 > 79 characters) Line 350:80: E501 line too long (81 > 79 characters) Line 415:80: E501 line too long (127 > 79 characters) Line 416:80: E501 line too long (109 > 79 characters) Line 433:80: E501 line too long (131 > 79 characters) Line 434:80: E501 line too long (119 > 79 characters)