Skip to content

Remove subscription_thread_dispatcher.h from public API - #265

Merged
stephen-derosa merged 2 commits into
mainfrom
sderosa/private-subscription-thread-dispatcher
Sep 23, 2026
Merged

stephen-derosa merged 2 commits into
mainfrom
sderosa/private-subscription-thread-dispatcher

Conversation

@stephen-derosa

Copy link
Copy Markdown
Collaborator

Overview

subscription_thread_dispatcher.h: SubscriptionThreadDispatcher was never meant to be in the public API.

This moves it to the LIVEKIT_INTERNAL_API and to the src/ folder.

For this change we will create a minor version bump since it was never intended to be in the public API to begin with.

@stephen-derosa stephen-derosa self-assigned this Sep 17, 2026
Copilot AI lite review requested due to automatic review settings September 17, 2026 20:19
devin-ai-integration[bot]

This comment was marked as resolved.

This comment was marked as resolved.

@stephen-derosa
stephen-derosa force-pushed the sderosa/private-subscription-thread-dispatcher branch 2 times, most recently from 9206faf to 01c0487 Compare September 17, 2026 20:45
devin-ai-integration[bot]

This comment was marked as resolved.

Comment thread include/livekit/room_callbacks.h
Comment thread include/livekit/e2ee.h Outdated
@stephen-derosa
stephen-derosa force-pushed the sderosa/private-subscription-thread-dispatcher branch from 8b5aff6 to 41b4d31 Compare September 23, 2026 20:25
@stephen-derosa
stephen-derosa merged commit e6e191b into main Sep 23, 2026
50 of 52 checks passed
@stephen-derosa
stephen-derosa deleted the sderosa/private-subscription-thread-dispatcher branch September 23, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants