-
Notifications
You must be signed in to change notification settings - Fork 44
chore(repo): Release v0.11.0 #1072
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
Conversation
WalkthroughBump workspace packages to 0.11.0, update stream_video-related dependency ranges, update changelogs to 0.11.0 with Android/Flutter/Dart requirements and breaking-change notes, and increment the streamVideoVersion constant in globals.dart. No runtime logic edits beyond version strings. Changes
Sequence Diagram(s)Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1072 +/- ##
=====================================
Coverage 4.80% 4.80%
=====================================
Files 591 591
Lines 40395 40395
=====================================
Hits 1941 1941
Misses 38454 38454 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Actionable comments posted: 2
🧹 Nitpick comments (5)
packages/stream_video_noise_cancellation/CHANGELOG.md (1)
7-11
: Use consistent comparison operators across itemsAGP/Gradle use “>=” while Kotlin is a plain “2.2.0”. Consider “Kotlin >= 2.2.0” (if older patch versions work) or “Kotlin = 2.2.0” if it must be pinned.
packages/stream_video_flutter/CHANGELOG.md (1)
18-27
: Add a short migration snippet forCall.stats
renameInclude a before/after code sample to reduce upgrade friction (mapping
publisherStats
→publisherStatsBundle
, etc.).+// Before +final stats = await call.stats.first; +final pub = stats.publisherStats; + +// After (0.11.0) +final stats = await call.stats.first; +final pub = stats.publisherStatsBundle;packages/stream_video/CHANGELOG.md (2)
18-27
: Clarify migration path for removed fieldsAdd a brief mapping table (old CallState fields → new surfaces), e.g.,
publisherStats/subscriberStats
→call.stats
stream, aggregated →call.statsReporter?.currentMetrics
.
7-11
: Android requirements consistency and completeness
- Rename “compileSDK 36” to “compileSdk 36.”
- Add “JDK 17+ required” to the requirements list.
- AGP 8.12.1 and Gradle 8.13 are released and compatible (AGP 8.x requires JDK 17).
packages/stream_video_noise_cancellation/pubspec.yaml (1)
16-18
: Alignplugin_platform_interface
version across pluginsThis package uses
plugin_platform_interface: ^2.0.2
, whilestream_video_screen_sharing
uses^2.1.8
. Consider bumping here to^2.1.8
to avoid duplicate transitive resolutions and keep SDK guidance consistent.- plugin_platform_interface: ^2.0.2 + plugin_platform_interface: ^2.1.8
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (14)
dogfooding/pubspec.yaml
(2 hunks)melos.yaml
(1 hunks)packages/stream_video/CHANGELOG.md
(1 hunks)packages/stream_video/lib/globals.dart
(1 hunks)packages/stream_video/pubspec.yaml
(1 hunks)packages/stream_video_flutter/CHANGELOG.md
(1 hunks)packages/stream_video_flutter/example/pubspec.yaml
(1 hunks)packages/stream_video_flutter/pubspec.yaml
(2 hunks)packages/stream_video_noise_cancellation/CHANGELOG.md
(1 hunks)packages/stream_video_noise_cancellation/pubspec.yaml
(2 hunks)packages/stream_video_push_notification/CHANGELOG.md
(1 hunks)packages/stream_video_push_notification/pubspec.yaml
(2 hunks)packages/stream_video_screen_sharing/CHANGELOG.md
(1 hunks)packages/stream_video_screen_sharing/pubspec.yaml
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
- GitHub Check: analyze
- GitHub Check: build
- GitHub Check: stream_video
- GitHub Check: stream_video_push_notification
- GitHub Check: stream_video_noise_cancellation
- GitHub Check: stream_video_flutter
🔇 Additional comments (12)
packages/stream_video/pubspec.yaml (1)
4-4
: LGTM! Version bump aligns with the 0.11.0 release.The version update is consistent with the coordinated release across the stream_video package ecosystem.
packages/stream_video_push_notification/CHANGELOG.md (1)
1-15
: LGTM! Well-documented breaking changes for the 0.11.0 release.The CHANGELOG entry clearly communicates the breaking changes for Android development with specific version requirements, making it easy for developers to understand what they need to update.
packages/stream_video_flutter/example/pubspec.yaml (1)
30-32
: LGTM! Dependency versions correctly updated for the 0.11.0 release.The example app dependencies are properly aligned with the new release version.
packages/stream_video/lib/globals.dart (1)
4-4
: LGTM! Version constant updated for the 0.11.0 release.This change ensures the SDK reports the correct version in user-agent strings and version checks.
packages/stream_video_screen_sharing/CHANGELOG.md (1)
1-15
: LGTM! Consistent breaking changes documentation for the 0.11.0 release.The CHANGELOG entry maintains consistency with other packages in the ecosystem and clearly documents the Android breaking changes and SDK requirements.
packages/stream_video_noise_cancellation/pubspec.yaml (1)
9-12
: Env constraints match changelog — LGTMDart ^3.8.0 and Flutter >=3.32.0 align with the 0.11.0 notes.
packages/stream_video_screen_sharing/pubspec.yaml (2)
5-5
: Version bump looks good0.10.4 → 0.11.0 with unchanged API surface.
10-12
: Env constraints consistentDart ^3.8.0 and Flutter >=3.32.0 match 0.11.0 requirements.
dogfooding/pubspec.yaml (1)
2-2
: Flutter SDK unavailable in sandbox—please verify locally
I couldn’t runflutter pub get
ordart analyze --fatal-infos .
here (Flutter isn’t installed). Please run those commands in your local environment to confirm there are no errors.packages/stream_video_push_notification/pubspec.yaml (1)
4-4
: Approve version bump and CHANGELOG entry
CHANGELOG.md contains a## 0.11.0
entry that matches the pubspec version.melos.yaml (1)
26-30
: Deps aligned to ^0.11.0; no ^0.10.x constraints found, global version constant updated.packages/stream_video_flutter/pubspec.yaml (1)
4-4
: Approve version bump to 0.11.0; verify intl constraintConfirm with Flutter 3.32 whether the
intl: <=0.21.0
cap in packages/stream_video_flutter/pubspec.yaml is still required to avoid unnecessary resolution conflicts.
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.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (2)
packages/stream_video/CHANGELOG.md
(1 hunks)packages/stream_video_flutter/CHANGELOG.md
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
- GitHub Check: build
- GitHub Check: analyze
- GitHub Check: stream_video_noise_cancellation
- GitHub Check: stream_video_flutter
- GitHub Check: analyze_legacy_version
- GitHub Check: stream_video
- GitHub Check: stream_video_push_notification
> - Minimum compileSDK 36 | ||
> - Android Gradle Plugin >=8.12.1 | ||
> - Gradle wrapper >=8.13 | ||
> - Kotlin 2.2.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.
🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
Please align the Android tooling requirements with the actual project configuration.
The changelog still lists “compileSDK 36”, “Android Gradle Plugin >=8.12.1”, and “Gradle wrapper >=8.13”, but the repository now uses compileSdk = 36
, AGP 8.13.0, and Gradle 9.1.0. Please fix the casing (compileSdk
), bump the AGP/Gradle minimums, and add the mandatory JDK 17 requirement so the docs accurately match the build setup.
To confirm the current versions in the repo, run:
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "Android Gradle Plugin version(s):"
rg -nP "com\.android\.tools\.build:gradle" -C2
echo -e "\nGradle wrapper distribution:"
fd gradle-wrapper.properties --type f --exec sh -c 'echo "==> $1"; cat "$1"' sh
Length of output: 1747
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "compileSdk version(s):"
rg -nP "compileSdk(Version)?\s*=\s*\d+" -C1
Length of output: 22
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "Searching for compileSdkVersion assignments:"
rg -nP "compileSdk(Version)?\s+\d+" -n
echo -e "\nInspecting stream_video_flutter build.gradle:"
FILE=$(fd --type f --full-path "packages/stream_video_flutter/android/build.gradle" || echo "")
if [ -n "$FILE" ]; then
echo "==> $FILE"
rg -nP "compileSdk|com.android.tools.build:gradle" -n "$FILE"
else
echo "stream_video_flutter build.gradle not found"
fi
Length of output: 44
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "All 'compileSdk' occurrences in repo:"
rg -n "compileSdk" -n || true
echo -e "\nGradle build files under stream_video_flutter:"
fd --type f --full-path "packages/stream_video_flutter/android" -e gradle -e kts
echo -e "\nInspect each for compileSdk and AGP:"
for file in $(fd --type f --full-path "packages/stream_video_flutter/android" -e gradle -e kts); do
echo "==> $file"
rg -n "compileSdk" -n "$file" || true
rg -n "com.android.tools.build:gradle" -n "$file" || true
done
Length of output: 910
Align changelog with actual Android tooling versions
CHANGELOG.md (lines 8–11): correct casing (compileSdk
), and list the real minimums—AGP ≥ 8.12.1 and Gradle wrapper ≥ 8.13 (compileSdkVersion is set via flutter.compileSdkVersion
).
🤖 Prompt for AI Agents
In packages/stream_video_flutter/CHANGELOG.md around lines 8 to 11, update the
Android tooling entries to use the correct casing and actual minimums: change
"compileSDK" to "compileSdk" and note that compileSdkVersion is controlled via
flutter.compileSdkVersion; ensure entries state "Android Gradle Plugin ≥ 8.12.1"
and "Gradle wrapper ≥ 8.13" exactly as the minimums.
Summary by CodeRabbit
Breaking Changes
New Features
Bug Fixes
Chores
Documentation