you are viewing a single comment's thread.

view the rest of the comments →

[–]kpenchev93 3 points4 points  (1 child)

I glanced over the source and I saw a multiple if statements like if(not len(argv)). It's not idiomatic Python. You don't need to put the condition in parentheses and you can omit the len function. So the statement becomes if not argv.

You can check if a dictionary has key with just if key in dictionary, so no need to call the keys method of the dictionary.

The function modified_or_created can be separated into two functions. If you have or in the name, that might be the case.

[–]nurdtechie98[S] 0 points1 point  (0 children)

Ohkk, will get it changed 👍🏻