Skip to content

Commit

Permalink
remove todo comment
Browse files Browse the repository at this point in the history
  • Loading branch information
elfkuzco committed Oct 15, 2024
1 parent d5ca336 commit 80f2c09
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/ted2zim/scraper.py
Original file line number Diff line number Diff line change
Expand Up @@ -1035,12 +1035,11 @@ def download_speaker_image(
try:
# Before downloading a speaker image, check if the URL exists.
# Sometimes, the URL from TED is "-" which is invalid.
# TODO: Implement a robust URL matcher if there are more invalid URLs
if not video_speaker:
logger.debug("Speaker doesn't have an image")
elif video_speaker == "-":
logger.error(

Check warning on line 1041 in src/ted2zim/scraper.py

View check run for this annotation

Codecov / codecov/patch

src/ted2zim/scraper.py#L1041

Added line #L1041 was not covered by tests
f"Invalid speaker image URL {video_speaker} for "
f"Invalid speaker image URL {video_speaker!r} for "
f"{video_title}"
)
else:
Expand Down

0 comments on commit 80f2c09

Please sign in to comment.