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.

Sign up for GitHub

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

Jump to bottom

Cache subtitles in S3 storage #287

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Update `download_subtitles` method to cache subtitles in S3 storage

d34a734
Select commit
Loading
Failed to load commit list.
Draft

Cache subtitles in S3 storage #287

Update `download_subtitles` method to cache subtitles in S3 storage
d34a734
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Aug 4, 2024 in 1s

0.00% of diff hit (target 1.54%)

View this Pull Request on Codecov

0.00% of diff hit (target 1.54%)

Annotations

Check warning on line 886 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L885-L886 

Added lines #L885 - L886 were not covered by tests

Check warning on line 889 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L888-L889 

Added lines #L888 - L889 were not covered by tests

Check warning on line 892 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L892 

Added line #L892 was not covered by tests

Check warning on line 900 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L899-L900 

Added lines #L899 - L900 were not covered by tests

Check warning on line 908 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L904-L908 

Added lines #L904 - L908 were not covered by tests

Check warning on line 915 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L913-L915 

Added lines #L913 - L915 were not covered by tests

Check warning on line 919 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L919 

Added line #L919 was not covered by tests

Check warning on line 922 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L922 

Added line #L922 was not covered by tests

Check warning on line 924 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L924 

Added line #L924 was not covered by tests

Check warning on line 930 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L928-L930 

Added lines #L928 - L930 were not covered by tests

Check warning on line 938 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L935-L938 

Added lines #L935 - L938 were not covered by tests

Check warning on line 948 in scraper/src/youtube2zim/scraper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scraper/src/youtube2zim/scraper.py#L948 

Added line #L948 was not covered by tests
View more details on Codecov