refactor: delete the dead StatusToForegroundValueConverter
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 1/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 91/100
調査の方向性
src/LageBuch.App.Shared/Converters/StatusToForegroundValueConverter.csを削除し、Converters/に何か残っているか確認します。空の場合はフォルダーを削除します。次に、報告された検索で見つからなかった参照を検出するため、make buildとmake testを実行します。両方のコマンドが成功し、未使用のconverterがなくなれば完了です。
索引モデルが issue の本文から書いたものです。
説明
src/LageBuch.App.Shared/Converters/StatusToForegroundValueConverter.cs is dead
code, and it is also broken code.
A repo-wide search for StatusToForeground across src/ and tests/, in both
.cs and .axaml files, matches only the declaration itself. Nothing binds to
it. On top of that its Convert ignores the value argument entirely and
always returns a black brush, which on this dark theme would be invisible if it
ever were used.
What to do
Delete the file. Check whether Converters/ still holds anything afterwards and
remove the folder if it does not.
Then build and run the tests to confirm nothing referenced it through a string
in XAML that a grep would miss.
Verify
make build
make test
Difficulty: easy, roughly ten minutes. A clean first pull request.
New here?
Comment to claim this issue and it is yours. If no pull request appears within
seven days it goes back in the pool, no hard feelings.
Before you open the PR, please skim CONTRIBUTING.md:
commits need DCO sign-off (git commit -s) and a Conventional Commit subject.
Changes under src/ additionally need an entry under ## [Unreleased] in
CHANGELOG.md — CI fails the PR otherwise.
Questions are welcome right here or in
Discussions.
- 主要言語
- C#
- スター
- 1
- フォーク
- 3
- 平均マージ
- 4時間 15分
- マージ済み PR（30日）
- 181
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CodeForFire/lagebuch のほかの issue
-
Stammdaten: in the Checkliste editor the PFLICHT header is not aligned with its column対応中かも @Saad-elnagar が 1 日前に担当しました。 オープンbug ux
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
CodeForFire/lagebuch#714 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Stammdaten: on a phone the Fahrzeuge editor squeezes Wache and Funkrufname to nothing対応中かも @Saad-elnagar が 2 日前に担当しました。 オープンbug ux
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
CodeForFire/lagebuch#713 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
CodeForFire/lagebuch#405 ·
メンテナーはふだん 1 日以内に返信
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
CodeForFire/lagebuch#349 ·
メンテナーはふだん 1 日以内に返信
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
CodeForFire/lagebuch#348 ·
メンテナーはふだん 1 日以内に返信
CodeForFire/lagebuch の issue をすべて見る
似ている issue
-
bug mcp misleading small
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
メンテナーはふだん 1 日以内に返信
-
accessibility help wanted macos
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
dorlugasigal/annotator#42 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
AvaloniaUI/Avalonia#22568 ·
メンテナーはふだん 2 日以内に返信
-
[bug] 截图失败日志冲爆硬盘オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 63/100
babalae/better-genshin-impact#3846 · コメント 2 件 ·
メンテナーはふだん 2 日以内に返信
-
[辞書] ブルアカ->ブルーアーカイブオープン要トリアージ 辞書
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信