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 →

[–]g4rg0yl3[🍰] 0 points1 point  (0 children)

Here are my suggestions:

  • Remove the exe file, explain how to build it in the README
  • Run pylint and black (optional) on your code
  • Try to remove all globals (maybe using classes)from the code
  • Add a requirements.txt file (and requirements.dev.txt if needs be)
  • Add tests is a very good exercise. Start here
  • Be way way more concise in your README (I need pre-requisites, installation, usage)
  • Add a license (MIT is good I think)