`dcm check-unused-code` false positives
まだ誰も着手していません。
評価
調査の方向性
まず dcm check-unused-code packages/devtools_app/ --exclude-public-api を実行し、報告された Dart ファイル(extent_delegate_utils.dart、_controller_web.dart、home_screen.dart、app.dart、_perfetto_controller_web.dart を含む)を調べます。リストされたフィールド、メソッド、クラスを、それらの assert での使用箇所まで追跡し、実際に使用されているものが報告されなくなっている一方で、本当に未使用のコードは引き続き報告されることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Command: dcm check-unused-code packages/devtools_app/ --exclude-public-api
False positives (one common theme I see here is that anything used in assert statements is falsely determined "unused"):
packages/devtools_app/test/test_infra/utils/extent_delegate_utils.dart (1 issue):
✖ unused field _renderObjectInitialized
at packages/devtools_app/test/test_infra/utils/extent_delegate_utils.dart:19:3
packages/devtools_app/lib/src/extensions/embedded/_controller_web.dart (1 issue):
✖ unused field _initialized
at packages/devtools_app/lib/src/extensions/embedded/_controller_web.dart:81:3
packages/devtools_app/lib/src/framework/home_screen.dart (2 issues):
✖ unused method _debugInitVmServiceCache
at packages/devtools_app/lib/src/framework/home_screen.dart:174:3
packages/devtools_app/lib/src/app.dart (2 issues):
✖ unused class _AlternateCheckedModeBanner
at packages/devtools_app/lib/src/app.dart:616:1
✖ unused constructor _AlternateCheckedModeBanner
at packages/devtools_app/lib/src/app.dart:622:3
packages/devtools_app/lib/src/screens/performance/panes/timeline_events/perfetto/_perfetto_controller_web.dart (1 issue):
✖ unused field _initialized
at packages/devtools_app/lib/src/screens/performance/panes/timeline_events/perfetto/_perfetto_controller_web.dart:162:3
I will add more false positives to this list as I find them (working through different directories https://github.com/flutter/devtools/issues/9906).
@incendial
- 主要言語
- 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
-
status: needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
a2ui-project/a2ui#2782 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
MaikuB/flutter_local_notifications#2832 · コメント 1 件 ·
-
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