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

Newtonsoft.Json への参照を package.json に追加 #10

Merged
merged 2 commits into from
Apr 15, 2024
Merged

Newtonsoft.Json への参照を package.json に追加 #10

merged 2 commits into from
Apr 15, 2024

Conversation

sus-taguchi-t
Copy link
Contributor

@sus-taguchi-t sus-taguchi-t commented Apr 15, 2024 •
edited
Loading

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

  • Newtonsoft.Json への参照を package.json に追加しました
  • 失敗したテストを修正しました

何を確認しましたか?

実装

参照方法の変更のみのため以下は未確認

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

テスト

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

変更影響

extreal-dev/Extreal.Guide#76

  • GuideのReleaseページに変更内容が追加されることを確認しました
  • 後方互換が崩れている場合はGuideのReleaseページのUpgrade guideに対応方法が追加されることを確認しました
  • GuideのModuleページ（機能ページ）に変更が反映されることを確認しました
  • GuideのLearningページに変更が反映されることを確認しました
  • Sample Applicationに変更が反映されることを確認しました

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

@sus-yoshikane-t sus-yoshikane-t merged commit cf36005 into develop Apr 15, 2024
1 check passed
@sus-yoshikane-t sus-yoshikane-t deleted the feature/change-newtonsoft-json branch April 15, 2024 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sus-yoshikane-t sus-yoshikane-t sus-yoshikane-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.
2 participants
@sus-taguchi-t @sus-yoshikane-t