Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

指定した関数呼び出しとコールバックのトレースログを抑制できるように変更 #7

Merged
merged 12 commits into from Feb 21, 2024
Merged

指定した関数呼び出しとコールバックのトレースログを抑制できるように変更 #7

merged 12 commits into from Feb 21, 2024

Conversation

sus-yoshikane-t
Copy link
Contributor

@sus-yoshikane-t sus-yoshikane-t commented Jan 11, 2024 •
edited

何の変更を加えましたか？

  • addAction/addFunction/callbackで指定したターゲット名を使用して、特定のトレースログを抑制できるように変更しました。

何を確認しましたか?

実装

  • Frameworkの誤った使い方にすぐに気づけるように、無効な引数や無効なメソッド呼び出しに対するチェックが入っていることを確認しました
  • Framework実行時の動きが分かるように、ログ（Error/Warn/Info/Debug）を出力していることを確認しました
  • 静的解析で問題が見つからないことを確認しました
  • フレームワーク利用者が使うAPI（主にprivate以外）に C# ドキュメントを記述しました

テスト

  • 全ての自動テストが成功することを確認しました
    • 自動テストなし
  • テストカバレッジが100%になることを確認しました
  • サンプルがあるものはサンプルが動作することを確認しました

変更影響

  • GuideのReleaseページに変更内容が追加されることを確認しました
  • GuideのModuleページ（機能ページ）に変更が反映されることを確認しました
  • GuideのLearningページに変更が反映されることを確認しました
    • 変更なし
  • Sample Applicationに変更が反映されることを確認しました
    • 変更なし

レビュアーへのメッセージ

@sus-yoshikane-t sus-yoshikane-t mentioned this pull request Jan 11, 2024
Merged
3 tasks
mahaweilu
mahaweilu reviewed Jan 11, 2024
View reviewed changes
WebScripts~/src/helper.ts Outdated Show resolved Hide resolved
WebScripts~/package.json Outdated Show resolved Hide resolved
@kiyohome kiyohome merged commit 84f1b10 into develop Feb 21, 2024
1 check passed
@kiyohome kiyohome deleted the trace-log-suppress branch February 21, 2024 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mahaweilu mahaweilu mahaweilu left review comments

@kiyohome kiyohome kiyohome approved these changes

@sus-taguchi-t sus-taguchi-t sus-taguchi-t approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@sus-yoshikane-t @kiyohome @mahaweilu @sus-taguchi-t @sasuke9410