DevLARLEY / LearnystDownloader

Downloads encrypted/unencrypted videos/audio/pdf/html/images from Learnyst
GNU General Public License v3.0
9 stars 7 forks source link

Fix Bug #3

Closed ThatNotEasy closed 1 month ago

ThatNotEasy commented 1 month ago
  1. Remove the Walrus Operator from the f-string: Instead of using the walrus operator (:=) within the f-string, assign the value to a variable before using it in the f-string.

  2. Use Context Managers: Ensure that files are properly managed and closed by using context managers (with open(...)).

  3. Reformat the Code: Update the code to assign values before using them in the logging message.

  4. The executable_exists function checks if an executable file exists in the binary/ directory. It searches for the specified file both with and without a .exe extension.

  5. Added config Folder: The configuration file is now placed in a newly created config directory, improving organization and separation of configuration data from other project files.

DevLARLEY commented 1 month ago

I've applied your suggested changes, but didn't merge it because there were still some things I didn't like.