chore(repo): bump min Flutter to 3.41.0 and Dart SDK to 3.11.0#2721
Conversation
Bumps the workspace minimum Flutter constraint from 3.38.1 to 3.41.9 and the Dart SDK floor from ^3.10.0 to ^3.11.5 across all pubspecs, melos.yaml, and the legacy_version_analyze CI pin. The legacy_version_analyze job (`dart analyze --fatal-infos lib/`) was failing on Flutter 3.38.1 because that analyzer flags deprecated-symbol references inside dartdoc comments (e.g. `[StreamSvgIcon]` in the StreamSvgIconData doc) as `deprecated_member_use_from_same_package`. Newer analyzers no longer report these. Aligning the legacy floor with the current N-1 stable resolves the failure. Co-Authored-By: Claude Opus 4.7 (1M context) <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 ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
…hangelogs - Lower minimum Flutter from 3.41.9 → 3.41.0 and Dart SDK from ^3.11.5 → ^3.11.0 across all pubspecs, melos.yaml, and the legacy_version_analyze CI pin. - Add CHANGELOG entries for stream_chat, stream_chat_flutter, stream_chat_flutter_core, stream_chat_persistence, and stream_chat_localizations. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/stream_chat/CHANGELOG.md`:
- Line 19: Update the changelog entry in CHANGELOG.md to include the minimum
Flutter version alongside the Dart SDK bump; change the line that currently
reads "Raised minimum Dart SDK to `^3.11.0`." to the consistent phrasing used
elsewhere e.g. "Raised minimum Flutter to `>=3.41.0` and Dart SDK to `^3.11.0`."
so the package entry matches stream_chat_persistence's format and includes both
platform requirements.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 7ea7a3e3-47d3-423f-9ad2-e6fe7f3c4bdc
⛔ Files ignored due to path filters (1)
pubspec.lockis excluded by!**/*.lock
📒 Files selected for processing (20)
.github/workflows/legacy_version_analyze.ymldocs/docs_screenshots/pubspec.yamlmelos.yamlpackages/stream_chat/CHANGELOG.mdpackages/stream_chat/example/pubspec.yamlpackages/stream_chat/pubspec.yamlpackages/stream_chat_flutter/CHANGELOG.mdpackages/stream_chat_flutter/example/pubspec.yamlpackages/stream_chat_flutter/pubspec.yamlpackages/stream_chat_flutter_core/CHANGELOG.mdpackages/stream_chat_flutter_core/example/pubspec.yamlpackages/stream_chat_flutter_core/pubspec.yamlpackages/stream_chat_localizations/CHANGELOG.mdpackages/stream_chat_localizations/example/pubspec.yamlpackages/stream_chat_localizations/pubspec.yamlpackages/stream_chat_persistence/CHANGELOG.mdpackages/stream_chat_persistence/example/pubspec.yamlpackages/stream_chat_persistence/pubspec.yamlpubspec.yamlsample_app/pubspec.yaml
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v10.0.0 #2721 +/- ##
========================================
Coverage 67.84% 67.84%
========================================
Files 411 411
Lines 24632 24632
========================================
Hits 16711 16711
Misses 7921 7921 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Summary
>=3.38.1to>=3.41.0and the Dart SDK floor from^3.10.0to^3.11.0across all 14 spots (rootpubspec.yaml,melos.yaml, every package + example pubspec, and thesample_app/docs_screenshotspubspecs).legacy_version_analyzeCI workflow pin from3.38.1to3.41.0to match.🔄 Changedentries forstream_chat,stream_chat_flutter,stream_chat_flutter_core,stream_chat_persistence, andstream_chat_localizations.Test plan
melos bootstrapsucceeds with the new constraintsmelos run analyze— No issues found across all 7 analyzed packagescd packages/<pkg>/lib && dart analyze --fatal-infos .— clean for all 5 library packages (locally on Dart 3.12.0; the real validation is thelegacy_version_analyzejob re-running on PR feat(repo)!: v10.0.0 #2499 with Flutter 3.41.0)legacy_version_analyzejob passes on PR feat(repo)!: v10.0.0 #2499 once this is merged intov10.0.0🤖 Generated with Claude Code
Summary by CodeRabbit