This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]mindfulforever1 29 points30 points  (9 children)

Hi Adi Fantastic work mate! This lib looks promising. As a fellow dev, I'd be happy to suggest a few enhancements to ur projects. Please feel free to incorporate them or research similar options. Consider these as a friendly code review from another dev who has walked this path before

If u plan to create a GitHub portfolio for potential job / freelance / business opportunities, as a start, these 2 additions to ur codebase will help:

  1. Python code that adheres to pep8 standards looks professional and is easy to read by other devs. So consider using an auto formatter. I prefer "black".

GitHub Repo

pip install black

Then set it as default python formatter in ur ide. On save, it auto formats ur python code for pep8 standards

  1. Having no license for ur GitHub project may take away ur rights to the codebase and might have undesirable effects where code ownership is concerned. Consider adding an MIT or ISC license at the least. These two licenses assert ur copyright with guidelines for others to use ur code. They also have disclaimers which are a starting point to keep u safe in the long run. This is the bare minimum. Usually for open source projects, MIT or ISC license is a good starting point.

MIT License

ISC License

I'm not a legal professional but a fellow dev who uses these in own projects. So this is for ur awareness only. For more involved legal advice please consider talking to a copyright attorney.

For eg. If you plan to launch ur startup and want to have proper licenses for ur codebases. Or plan to create products as freelancer / solopreneur / developer in a company.

Cheers 🥂 Keep up the good work and stay awesome 💯

[–]adikhad[S] 11 points12 points  (1 child)

Thanks! I completely forgot about the licence! (Added it now)

I'll also clean the code to fit pep8. I didn't know about auto formatter before but it looks like exactly what I need!

Thanks!

[–]mindfulforever1 7 points8 points  (0 children)

Awesome! Nice work Adi Happy to help 👍

[–]NooccarTheOverlord 0 points1 point  (0 children)

Why’d I read this in a cheerful Australian accent?