Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JS] Parallelise Downloads, Extract Image Names, Modern JS, General Refactor #51

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

optionallychained
Copy link

Just had a bit of fun with this, hope that's ok. Bit of a refactor to the JS version for a number of benefits:

  • significantly increase speed by downloading images in parallel
  • nicer file names extracted from asset URLs
  • nicer terminal output
  • readability / clarity / usage of modern JS features

@optionallychained optionallychained changed the title Improve JS version Parallelise Node Downloads + General JS Refactor Sep 26, 2024
@optionallychained optionallychained changed the title Parallelise Node Downloads + General JS Refactor [JS] Parallelise Downloads, Extract Image Names, General Cleanup Sep 26, 2024
@optionallychained optionallychained changed the title [JS] Parallelise Downloads, Extract Image Names, General Cleanup [JS] Parallelise Downloads, Extract Image Names, Modern JS, General Refactor Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant