dmarx / dmarx/VideoLinkBot

Refactor build_comment() so we don't waste time getting titles for videos that won't be listed due to the character limit

Open
#34 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
26
Forks
5
PR merge metrics
No merged PRs in 30d

Description

Instead of getting all video titles then formatting the links and appending to the comment body, format links and append them immediately after getting the video title. This way we can cut out as soon as the comment hits the character limit. This should also eliminate the need for the trim_comment() function, since we'll stop building comments before they hit the limit.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.