upgrade flutter dependencies#2867
Conversation
There was a problem hiding this comment.
Code Review
This pull request upgrades several Flutter dependencies, most notably pasteboard to version 0.5.0. My review focuses on the implications of these upgrades. I've identified a potential improvement opportunity regarding the pasteboard dependency, which appears to be unused and could possibly be removed.
| meta: ^1.17.0 | ||
| package_info_plus: ^9.0.0 | ||
| pasteboard: ^0.4.0 | ||
| pasteboard: ^0.5.0 |
There was a problem hiding this comment.
The pasteboard dependency appears to be unused. The project's clipboard functionality (e.g., in copy_moves_shortcut.dart and paste_moves_shortcut.dart) relies on the standard Clipboard API from flutter/services.dart, which handles text operations across all platforms, including macOS. The pasteboard package is generally for handling non-text data like images. Since it's not being used, consider removing this dependency to simplify the project's dependencies.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2867 +/- ##
=======================================
Coverage 94.11% 94.11%
=======================================
Files 52 52
Lines 1479 1479
=======================================
Hits 1392 1392
Misses 87 87
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Auto-generated by create-pull-request
See: https://github.com/peter-evans/create-pull-request/blob/master/docs/concepts-guidelines.md#triggering-further-workflow-runs