CommunityToolkit / CommunityToolkit/Windows

Validate BehaviorBase<T> and what is needed (if anything) beyond Behavior<T>

オープン
#27 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
components::behaviors need more info :pushpin: question
主要言語
C#
スター
1.1k
フォーク
166
PR マージ指標
30日以内にマージされた PR はありません

説明

We probably want to file an issue about re-checking all this, as I know some of the original issues I believe were fixed in the underlying behaviors package, but I think we still do some extra logic here.

Probably a thing where when we can better test these and shore that up, we could try removing the intermediate class and see what happens or if anything breaks. (Assuming we have enough coverage)

_Originally posted by @michael-hawker in https://github.com/CommunityToolkit/Windows/pull/20#discussion_r1167170975_

Related Toolkit Discussion: https://github.com/CommunityToolkit/WindowsCommunityToolkit/issues/3856

with specifically more prior context and links here: https://github.com/CommunityToolkit/WindowsCommunityToolkit/issues/3856#issuecomment-801471909

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

関連するToolkit Discussionとリンクされた以前のコンテキストから始め、次にBehaviorBaseとBehaviorを比較して、ここに残っている追加のロジックを特定します。動作を壊さずに中間クラスを削除できるか評価する前に、十分なテストカバレッジを確立してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
csharp
領域
developer-experience
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
20/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。