Skip to content

Add SeekableLiveConfig to the PillarboxBuilder#1180

Merged
StaehliJ merged 1 commit intomainfrom
add-seekable-range-to-pillarbox-builder
Oct 2, 2025
Merged

Add SeekableLiveConfig to the PillarboxBuilder#1180
StaehliJ merged 1 commit intomainfrom
add-seekable-range-to-pillarbox-builder

Conversation

@StaehliJ
Copy link
Contributor

@StaehliJ StaehliJ commented Oct 2, 2025

Description

Add to public api SeekableLiveConfig if the default configuration doesn't suits the needs.

Changes made

  • Self-explanatory

Checklist

  • APIs have been properly documented (if relevant).
  • The documentation has been updated (if relevant).
  • New unit tests have been written (if relevant).
  • The demo has been updated (if relevant).

@github-project-automation github-project-automation bot moved this to 📋 Backlog in Pillarbox Oct 2, 2025
@StaehliJ StaehliJ moved this from 📋 Backlog to 🍿 Code Review in Pillarbox Oct 2, 2025
@StaehliJ StaehliJ requested a review from waliid October 2, 2025 07:16
@StaehliJ StaehliJ enabled auto-merge October 2, 2025 07:16
@github-actions
Copy link

github-actions bot commented Oct 2, 2025

Code Coverage

Overall Project 47.72% -0.01% 🟢
Files changed 89.29% 🟢

Module Coverage
:pillarbox-player 56.28% -0.02% 🟢
Files
Module File Coverage
:pillarbox-player PillarboxBuilder.kt 78.55% -0.99% 🟢

@StaehliJ StaehliJ added this pull request to the merge queue Oct 2, 2025
Merged via the queue into main with commit 3a78e7b Oct 2, 2025
36 of 39 checks passed
@StaehliJ StaehliJ deleted the add-seekable-range-to-pillarbox-builder branch October 2, 2025 13:21
@github-project-automation github-project-automation bot moved this from 🍿 Code Review to ✅ Done in Pillarbox Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@waliid waliid waliid approved these changes

Assignees

No one assigned

Labels

None yet

Projects

Pillarbox
Status: ✅ Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@StaehliJ @waliid