Conversation
|
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 You can disable this status message by setting the 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Pull Request Overview
This PR standardizes dependency versions across the workspace by aligning minor and patch versions for core packages and dev tools.
- Bumped key dependencies (flutter_bloc, equatable, http, intl, decimal, args, path) to uniform versions
- Updated test and build_runner versions in dev_dependencies to ^1.25.7 and ^2.4.14 respectively
- Fixed missing caret on version constraints (e.g., equatable in komodo_cex_market_data)
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| products/komodo_compliance_console/pubspec.yaml | flutter_bloc 9.1.0 → 9.1.1 |
| products/dex_dungeon/pubspec.yaml | equatable 2.0.5 → 2.0.7, intl 0.20.0 → 0.20.2 |
| packages/komodo_wallet_cli/pubspec.yaml | test 1.25.2 → 1.25.7 |
| packages/komodo_wallet_build_transformer/pubspec.yaml | args 2.5.0 → 2.6.0, http 1.2.2 → 1.4.0, path 1.9.0 → 1.9.1, test 1.25.2 → 1.25.7 |
| packages/komodo_ui/pubspec.yaml | decimal 3.0.2 → 3.2.1, index_generator 4.0.0 → 4.0.1 |
| packages/komodo_defi_types/pubspec.yaml | equatable 2.0.5 → 2.0.7, meta 1.9.1 → 1.15.0, test 1.25.2 → 1.25.7 |
| packages/komodo_defi_sdk/pubspec.yaml | http 1.3.0 → 1.4.0, test 1.25.2 → 1.25.7 |
| packages/komodo_defi_sdk/example/pubspec.yaml | flutter_bloc 9.0.0 → 9.1.1 |
| packages/komodo_defi_rpc_methods/pubspec.yaml | decimal 3.0.2 → 3.2.1, equatable 2.0.5 → 2.0.7, test 1.24.0 → 1.25.7 |
| packages/komodo_defi_local_auth/pubspec.yaml | flutter_bloc 9.0.0 → 9.1.1 |
| packages/komodo_defi_framework/pubspec.yaml | http 1.2.2 → 1.4.0, build_runner 2.4.11 → 2.4.14 |
| packages/komodo_coins/pubspec.yaml | equatable 2.0.5 → 2.0.7, http 1.2.2 → 1.4.0 |
| packages/komodo_coin_updates/pubspec.yaml | flutter_bloc 9.1.0 → 9.1.1, http 1.3.0 → 1.4.0, test 1.24.0 → 1.25.7 |
| packages/komodo_cex_market_data/pubspec.yaml | equatable 2.0.7 → ^2.0.7, test 1.24.0 → 1.25.7 |
Comments suppressed due to low confidence (1)
packages/komodo_defi_sdk/pubspec.yaml:48
- [nitpick] The commented-out test dependency duplicates the newly added test entry. Consider removing this commented line to avoid confusion.
# test: ^1.25.7
|
Visit the preview URL for this PR (updated for commit 8f08882): https://komodo-defi-sdk--pr74-codex-standardise-pa-khasoj4k.web.app (expires Wed, 11 Jun 2025 14:04:16 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 7f9f5ac39928f333b6e8fcefb7138575e24ed347 |
As per Flutter guidelines, `pubspec.lock` shouldn’t be committed for packages, but this does not apply to the example apps within the SDK project. This is useful for keeping track of which version the direct/transistive dependencies are resolving to.
|
Addresses @DeckerSU's feedback on the latest SDK release, indicating inconsistent usage of package version constraints. For all dependencies used in multiple projects, the version constraints have been unified and set to the highest value shared by them. It is worth looking into if our version constraints are overly restrictive to the point where we are losing out on developers adding the package to their existing projects. |
Summary
Testing
flutter pub get --offlineflutter analyzedart format -o none .https://chatgpt.com/codex/tasks/task_e_683f2677fc148326bf4ef07d6b72b66d