Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

StopAsyncの bug-fix #167 #168

Closed
wants to merge 1 commit into from

Conversation

i-maruyama
Copy link
Contributor

@i-maruyama i-maruyama commented May 13, 2021

Issue 番号 / Issue ID

目的 / Purpose

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

検証方法 / How to test

  • Debug_Mock ビルドの UnitTest (こちらでは確認済)

コードの入手 / Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
dotnet restore

その他 / Other information

@keiji keiji self-requested a review May 13, 2021 13:18
Copy link
Collaborator

@keiji keiji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

コードベースの確認のみになります。
正常動作を確認するためにユニットテストを作成する選択肢もありますが、動作についてはコードから明らかであるので、まずは修正を取り込み、ユニットテストについては別Issueとするのが良いと考えています。

@keiji keiji requested a review from cocoa-dev May 13, 2021 14:45
@i-maruyama
Copy link
Contributor Author

コード自体は、PR #191 に取り込まれました。ユニットテストへの言及も、PR #191 に転記します。
#168 (review)

close しておきます。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debug_Mock での StopAsyncの bug-fix
2 participants