Repository navigation
chore(repo): resolve and analyze on Flutter 3.50 - #1616
Conversation
Allow freezed 4, needed for pub get to resolve on Flutter 3.50, and ignore the material/cupertino import deprecations in the webrtc example, which webrtc:analyze runs with --fatal-infos. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the
|
SDK Size
Measured on |
Size BreakdownWhat each SDK adds to an empty release app, per Dart package and native library. Dart sizes are uncompressed symbol sizes. Open the This run does not measure ios. Measured on android ·
|
| Component | Size | Δ vs v2 |
|---|---|---|
native:libjingle_peerconnection_so.so |
11.96 MB | 0 |
package:stream_video |
1.11 MB | 0 |
classes.dex |
261.9 KB | 0 |
dart:@unknown |
260.1 KB | 0 |
dart:_http |
154.1 KB | 0 |
package:protobuf |
146.0 KB | 0 |
dart:mixin_deduplication |
143.8 KB | 0 |
native:libdartjni.so |
121.7 KB | 0 |
dart:io |
99.3 KB | 0 |
dart:async |
75.6 KB | 0 |
package:stream_core |
69.5 KB | 0 |
package:stream_webrtc_flutter |
55.3 KB | 0 |
package:rxdart |
54.3 KB | 0 |
package:dio |
53.2 KB | 0 |
dart:core |
52.2 KB | 0 |
| other (71) | 537.7 KB | 0 |
android · stream_video_flutter: 17.41 MB (0)
| Component | Size | Δ vs v2 |
|---|---|---|
native:libjingle_peerconnection_so.so |
11.96 MB | 0 |
package:stream_video |
1.16 MB | 0 |
package:flutter |
699.1 KB | 0 |
classes.dex |
454.8 KB | 0 |
dart:@unknown |
434.9 KB | 0 |
dart:mixin_deduplication |
362.7 KB | 0 |
package:stream_video_flutter |
205.7 KB | 0 |
dart:_http |
154.1 KB | 0 |
package:protobuf |
145.2 KB | 0 |
dart:io |
136.5 KB | 0 |
package:stream_core_flutter |
124.2 KB | 0 |
native:libdartjni.so |
121.7 KB | 0 |
assets |
118.5 KB | 0 |
package:hive_ce |
99.2 KB | 0 |
dart:@shared |
84.2 KB | 0 |
| other (99) | 1.22 MB | 0 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v2 #1616 +/- ##
=======================================
Coverage 66.05% 66.05%
=======================================
Files 467 467
Lines 27361 27361
=======================================
Hits 18072 18072
Misses 9289 9289 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
🎯 Goal
Make v2 resolve and pass analysis on Flutter 3.50 (tested on 3.50.0-0.4.pre), so the beta check and a later Flutter bump don't break CI.
🛠 Implementation details
stream_videodev dependencyfreezednow allows 4.x (">=3.0.0 <5.0.0"). On 3.50flutter_testpinstest_api 0.7.14, so the webrtc packages that depend ontestneedtest >=1.31.2, which needsanalyzer >=13. freezed 3 tops out at analyzer 12, sopub getfailed. On the current Flutter (3.47.x) the lock file is unchanged and freezed 3 is still used, so generated files don't change.webrtc/stream_webrtc_flutter/example: ignoredeprecated_member_use. Flutter 3.50 deprecates thematerial.dart/cupertino.dartimports, andwebrtc:analyzeruns with--fatal-infos. The SDK packages are analyzed with--no-fatal-warnings, so their deprecation infos don't fail CI.The
deprecated_member_useignore is reverted as part of FLU-922, the migration tomaterial_ui/cupertino_ui.🧪 Testing
With Flutter 3.50.0-0.4.pre:
flutter pub get,melos run webrtc:analyze,melos run analyze:errorandmelos run format:verifypass. With 3.47.4:flutter pub getpasses with no lock file changes.🤖 Generated with Claude Code