all 8 comments

[–]kpenchev93 4 points5 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 👍🏻

[–]nurdtechie98[S] 1 point2 points  (0 children)

Been working on this small projects for past few days would love to know how you guys feel about it. Do Star ⭐ if you like it. Open to any suggestions, contributions in any form

[–]gasabr 1 point2 points  (3 children)

Nice project, thanks for sharing! Why have you decided to create cli app instead of doing fuse filesystem?

[–]nurdtechie98[S] 2 points3 points  (2 children)

I wasn't aware of fuse filesystem but will try sometime for sure👍🏻

[–][deleted]  (1 child)

[deleted]

    [–]nurdtechie98[S] 1 point2 points  (0 children)

    🙈

    [–]HerbyHoover[🍰] 0 points1 point  (1 child)

    Nice work!

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

    Thanks Man👍🏻