`dcm check-unused-code`: bugs discovered where code is incorrectly unused.
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 48/100
調査の方向性
まず dcm check-unused-code を実行し、inbound_references_tree.dart、vm_service_private_extensions.dart、cpu_profiler_controller.dart、treemap.dart で報告された箇所を調査します。各検出結果が本当に未使用のメンバーなのか、意図されたシリアライゼーション経路なのか、それともアプリ起動プロファイルの読み込みにおけるリグレッションなのかを判断します。各検出結果について、正当な削除または修正が行われていれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Findings:
- vm_developer screen: this to delete the unused code requires deleting most of this file. It seems like a bug that the description is not used. CC @bkonyi
packages/devtools_app/lib/src/screens/vm_developer/object_inspector/inbound_references_tree.dart (1 issue):
✖ unused field description
at packages/devtools_app/lib/src/screens/vm_developer/object_inspector/inbound_references_tree.dart:29:3
- vm_developer screen: it seems like a bug if we are intending
ProfileCodeto be serializable but it is unused. CC @bkonyi
packages/devtools_app/lib/src/screens/vm_developer/vm_service_private_extensions.dart (8 issues):
✖ unused method toJson
at packages/devtools_app/lib/src/screens/vm_developer/vm_service_private_extensions.dart:856:3
- profiler screen: we used to have functionality to load an app startup profile by a user tag. This should be investigated to see if this method can be removed or if the fact that this is unused is a regression.
packages/devtools_app/lib/src/screens/profiler/cpu_profiler_controller.dart (1 issue):
✖ unused method loadAppStartUpProfile
at packages/devtools_app/lib/src/screens/profiler/cpu_profiler_controller.dart:414:3
- app size screen
packages/devtools_app/lib/src/shared/charts/treemap.dart (1 issue):
✖ unused field childrenMap
at packages/devtools_app/lib/src/shared/charts/treemap.dart:685:3
- 主要言語
- Dart
- スター
- 1.7k
- フォーク
- 405
- 平均マージ
- 4日 21時間
- マージ済み PR（30日）
- 15
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
flutter/devtools のほかの issue
-
cleanup P1
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
-
a: quality cost: low P2
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
contributions welcome cross-screen inspector-2.0-polish P2
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
cross-screen P3
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
documentation good first issue P2
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
flutter/devtools の issue をすべて見る
似ている issue
-
bug triage
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
fluttercommunity/plus_plugins#3979 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
from.page-issue
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
enhancement UI: Material
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
Notsfsssf/pixez-flutter#1360 ·
-
triage
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100