Thank you for your contribution @sxiii I am sure this is useful for a movie fan.
- I find no comments
- The commit messages could be more descriptive.
- all the three files could be extracted to a common class e.g.
MovieDBso that you can pass different command line options instead of 3 files with basically similar logics. Also theMovieDBclass could be customised with different parameters. - You could consider adding tests
- It would be good to support parameters like returning the TOP 10 popular movies by ratings or date.
Thanks!
Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.
To view those questions and the relevant answers related to your post, click here.
Need help? Chat with us on Discord.
@justyy thanks. This is an idea from my mind which was realized and rewritten few times already. I hope it will get comments soon. I totally agree with your idea on making it a common class! And I will look forward to add tests and making it more like a CLI tool if I see that this project will be used but someone on github. Thanks again for your in-depth review! Maybe someone could help me a little with python =)))
Thank you for your review, @justyy! Keep up the good work!