Skip to content

Pin the video SDK to the released WebRTC 145.19.0 - #1883

Merged
aleksandar-apostolov merged 1 commit into
develop-v2from
chore/v2-webrtc-145.19.0
Oct 20, 2026
Merged

aleksandar-apostolov merged 1 commit into
develop-v2from
chore/v2-webrtc-145.19.0

Conversation

@PratimMallick

@PratimMallick PratimMallick commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Goal

Closes AND-1677 — depend on the released stream-video-webrtc-android-repackaged 145.19.0 instead of the snapshot.

Implementation

  • Set streamWebRTC in gradle/libs.versions.toml from 145.19.0-SNAPSHOT to 145.19.0.
  • 145.19.0 is the current release on Maven Central. No other catalog entry is a snapshot.

Testing

  • ./gradlew :stream-video-android-core:dependencies --configuration releaseRuntimeClasspath resolves io.getstream:stream-video-webrtc-android-repackaged:145.19.0.
  • ./gradlew :stream-video-android-core:compileDebugKotlin :stream-video-android-core:apiCheck passed.
  • The full ./gradlew spotlessCheck detekt apiCheck testDebugUnitTest was not run. The diff is a version string only, and the API check covers the WebRTC surface core compiles against.

Made with Cursor

Summary by CodeRabbit

  • Chores
    • Updated the streamWebRTC dependency to the stable 145.19.0 release.

@PratimMallick PratimMallick added the pr:dependencies Dependency updates label Oct 20, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR checklist ✅

All required conditions are satisfied:

  • Title length is OK (or ignored by label).
  • At least one pr: label exists.
  • Sections ### Goal, ### Implementation, and ### Testing are filled, or the PR is bot-authored.
  • An issue is linked (Linear ticket or GitHub issue), or the PR is bot-authored.

🎉 Great job! This PR is ready for review.

@PratimMallick

Copy link
Copy Markdown
Contributor Author

@CodeRabbit review

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@PratimMallick
PratimMallick marked this pull request as ready for review October 20, 2026 08:27
@PratimMallick
PratimMallick requested a review from a team as a code owner October 20, 2026 08:27
@PratimMallick PratimMallick added the pr:internal Internal or infra-only changes label Oct 20, 2026
@coderabbitai

coderabbitai Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: bdc15230-838e-4c60-aed5-06eafb8cdcb2
📥 Commits

Reviewing files that changed from the base of the PR and between f192106 and f1db2d6.

📒 Files selected for processing: 1
  • gradle/libs.versions.toml

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

📝 Walkthrough

The version catalog changes the streamWebRTC dependency from 145.19.0-SNAPSHOT to 145.19.0.

Suggested reviewers: aleksandar-apostolov

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to f1db2

No actionable merge-blocking risk remains from this dependency version update.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly and concisely describes the primary change: pinning the video SDK to released WebRTC 145.19.0.
Description check Passed The description includes the Goal, Implementation, and Testing sections. It identifies the linked issue, the exact dependency change, and the validation performed. The template Checklist section is om…
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description includes the Goal, Implementation, and Testing sections. It identifies the linked issue, the exact dependency change, and the validation performed. The template Checklist section is omitted, but the description is otherwise substantially complete.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit checks the version line,
The snapshot hops away,
A release number takes its place,
The catalog looks aligned,
And carrots crunch to celebrate.

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

SDK Size Comparison 📏

SDK Before After Difference Status
stream-video-android-core 12.38 MB 12.66 MB 0.29 MB 🟡
stream-video-android-ui-xml 5.66 MB 5.70 MB 0.04 MB 🟢
stream-video-android-ui-compose 6.19 MB 3.16 MB -3.03 MB 🚀

@sonarqubecloud

Copy link
Copy Markdown

@rahul-lohra rahul-lohra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@aleksandar-apostolov
aleksandar-apostolov merged commit 88d73e1 into develop-v2 Oct 20, 2026
19 of 20 checks passed
@aleksandar-apostolov
aleksandar-apostolov deleted the chore/v2-webrtc-145.19.0 branch October 20, 2026 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:dependencies Dependency updates pr:internal Internal or infra-only changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants