-
Notifications
You must be signed in to change notification settings - Fork 70
chore: bump webrtc to m150 and enable WARP. #461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -74,7 +74,7 @@ | |
| "@eslint/eslintrc": "^3.3.1", | ||
| "@eslint/js": "^9.35.0", | ||
| "@livekit/changesets-changelog-github": "^0.0.4", | ||
| "@livekit/react-native-webrtc": "^144.2.0", | ||
| "@livekit/react-native-webrtc": "https://github.com/livekit/react-native-webrtc.git#bump-libwebrtc-to-m150", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🟡 Stale peer range admits incompatible WebRTC The declared Learn moreThe npm peer dependency defines which adapter versions consumers may install. The new wrapper passes Example: An app already using Recommended fix: Release the m150 adapter under a distinct npm version, update Was this helpful? React with 👍 or 👎 to provide feedback. |
||
| "@react-native/babel-preset": "0.83.0", | ||
| "@react-native/eslint-config": "0.83.0", | ||
| "@types/fastestsmallesttextencoderdecoder": "^1.0.0", | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟡 Private pod source never reaches consumers
External iOS apps do not inherit these
sourcedeclarations. Their pod installation cannot resolve LiveKitWebRTC 150 outside CocoaPods trunk.Learn more
CocoaPods resolves spec repositories from the application Podfile, not from Podfiles shipped elsewhere in an npm package. These declarations fix only the repository's CI app and example app. The adapter podspec pins
LiveKitWebRTC150.7871.02, which this change states is unavailable on trunk. A consuming application using the documented installation flow therefore has no source containing that podspec.Example: An existing app upgrades
@livekit/react-native, keeps its standard trunk-only Podfile, and runspod install. CocoaPods reports that no spec satisfiesLiveKitWebRTC (= 150.7871.02)instead of installing the SDK.Recommended fix: Publish LiveKitWebRTC 150 to trunk before releasing this package, or document and automate adding
https://github.com/livekit/podspecs.gitin every supported consumer integration, including the Expo plugin.Was this helpful? React with 👍 or 👎 to provide feedback.