Skip to content
This repository was archived by the owner on Apr 17, 2023. It is now read-only.

Update VideonDownloader download logic to download in parallel only when its available #46

Merged
merged 1 commit into from
Dec 4, 2022

Conversation

code-yeongyu
Copy link
Owner

  • fix YoutubeDLWrapper downloading
  • Update VideoDownloader download logic

This commit changes download logic.
Parallel downloading is enabled only when given link is multiple.
@codecov-commenter
Copy link

Codecov Report

Merging #46 (6d90783) into master (7783e35) will decrease coverage by 0.56%.
The diff coverage is 22.22%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
- Coverage   35.38%   34.82%   -0.57%     
==========================================
  Files          11       11              
  Lines         308      313       +5     
==========================================
  Hits          109      109              
- Misses        199      204       +5     
Impacted Files Coverage Δ
twitter_video_tools/video_downloader.py 46.15% <22.22%> (-10.99%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@code-yeongyu code-yeongyu merged commit 29beba3 into master Dec 4, 2022
@code-yeongyu code-yeongyu deleted the feature/video-downloader-parallel branch December 4, 2022 09:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants