Skip to content

Implement batch translation feature#9

Merged
jdfalk merged 1 commit intomainfrom
codex/review-project-documentation-and-implement-feature
Jun 7, 2025
Merged

Implement batch translation feature#9
jdfalk merged 1 commit intomainfrom
codex/review-project-documentation-and-implement-feature

Conversation

@jdfalk
Copy link
Copy Markdown
Owner

@jdfalk jdfalk commented Jun 7, 2025

Summary

  • add batch command for translating multiple subtitle files concurrently
  • extract translation helpers to pkg/subtitles
  • document batch translation in README, TODO, design docs, and CHANGELOG
  • update translate command to use helper
  • add tests for translation helpers

Testing

  • go test ./...

https://chatgpt.com/codex/tasks/task_e_6843abc8b69c8321b155298fcee3d6f1

@jdfalk jdfalk added the codex Created or modified by AI/automation agents label Jun 7, 2025 — with ChatGPT Codex Connector
@jdfalk jdfalk merged commit 1ad4bd3 into main Jun 7, 2025
@jdfalk jdfalk deleted the codex/review-project-documentation-and-implement-feature branch June 7, 2025 15:11
@jdfalk jdfalk added the duplicate This issue or pull request already exists label Jun 12, 2025
@jdfalk
Copy link
Copy Markdown
Owner Author

jdfalk commented Jun 12, 2025

Duplicate of #3

@jdfalk jdfalk marked this as a duplicate of #3 Jun 12, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Caching has partial implementation:

✅ Workflow-level caching:

  • Python dependencies cached
  • GitHub Actions cache for artifacts

❌ Still needed:

  • API response caching
  • Issue state caching
  • GraphQL implementation for better performance

Should remain open for the application-level caching implementation.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 3, 2025

Caching has partial implementation:

✅ Workflow-level caching:

  • Python dependencies cached
  • GitHub Actions cache for artifacts

❌ Still needed:

  • API response caching
  • Issue state caching
  • GraphQL implementation for better performance

Should remain open for the application-level caching implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex Created or modified by AI/automation agents duplicate This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant