Skip to content

feat: condensed storage for subtitles and automatic captions - #1555

Open
tcely wants to merge 4 commits into
mainfrom
tcely-issue-1554-compact-exts
Open

feat: condensed storage for subtitles and automatic captions#1555
tcely wants to merge 4 commits into
mainfrom
tcely-issue-1554-compact-exts

Conversation

@tcely

@tcely tcely commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Fixes #1554

Lint complaints

  • E741: Use a better font. 🤔
sync/utils.py:
   85:5  PLR1730 Replace `if` statement with `scaled_width = max(scaled_width, width)`
  178:8  SIM118 Use `key in dict` instead of `key in dict.keys()`
  223:16 SIM118 Use `key in dict` instead of `key in dict.keys()`
  227:12 SIM118 Use `key in dict` instead of `key in dict.keys()`
  274:12 SIM118 Use `key in dict` instead of `key in dict.keys()`
  277:21 SIM118 Use `key in dict` instead of `key in dict.keys()`
  284:12 SIM118 Use `key in dict` instead of `key in dict.keys()`
  316:18 FURB188 [*] Prefer `str.removesuffix()` over conditionally replacing with slice.
  318:18 FURB188 [*] Prefer `str.removesuffix()` over conditionally replacing with slice.
  320:18 FURB188 [*] Prefer `str.removesuffix()` over conditionally replacing with slice.

@tcely tcely added this to the v1.0 milestone Aug 16, 2026
@tcely
tcely requested a review from meeb August 16, 2026 09:40
@tcely tcely self-assigned this Aug 16, 2026
@tcely
tcely marked this pull request as ready for review August 16, 2026 09:41
@tcely tcely moved this to In Progress in Status Aug 16, 2026
meeb
meeb previously approved these changes Aug 16, 2026
@github-project-automation github-project-automation Bot moved this from In Progress to Ready in Status Aug 16, 2026
@tcely
tcely requested a review from meeb August 16, 2026 10:06
@tcely
tcely dismissed meeb’s stale review August 17, 2026 17:20

Approved while tests were failing.

@tcely
tcely force-pushed the tcely-issue-1554-compact-exts branch from 68673ea to ec7a386 Compare August 18, 2026 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready

Development

Successfully merging this pull request may close these issues.

Reduce storage costs for automatic captions and subtitles

2 participants