Open Kenny164 opened 8 years ago
I'm don't have merge rights but I thought I would help by reviewing this.
Curse has since changed the pack_base_url so it doesn't work now, but it will if you update it to:
pack_base_url = 'https://minecraft.curseforge.com/projects/%s/files/latest' % project_id
Other than that the code works for me, and looks good, thanks Kenny164. I reckon you should merge it after that change.
e.g.
downloader.py --project 242462
will get all the mods (at this time of writing).Apologies in advance for my code (it's been a while) and I'm not well versed in PRs nor python. This is more for advice about what I'm wanting to add on the long term.
I'm planning a couple more features such as:
At what point should I consider a new .py file and import your doDownload() function?
Thanks for creating this, making it available and your time.