-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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.
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
refactor/perf(backend): provide metadata statically #14601
Conversation
api.jsonの差分作成中にエラーが発生しました。詳細はWorkflowのログを確認してください。 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14601 +/- ##
===========================================
- Coverage 41.47% 39.65% -1.82%
===========================================
Files 1545 1541 -4
Lines 197907 192171 -5736
Branches 3595 2520 -1075
===========================================
- Hits 82077 76207 -5870
- Misses 115268 115400 +132
- Partials 562 564 +2 ☔ View full report in Codecov by Sentry. |
思ってたよりパフォーマンス向上効果ありそうな感じがしている |
あとサーバー設定の反映が一瞬で行われるようになった |
jestでどうmockすればいいのか分からん |
|
TASUKETE |
factoryは一回だけ呼ばれるから参照を保ったままにしないとダメだ |
謎 |
なぜかアンテナのe2eテストが失敗する |
ゔわー |
謎 |
別のe2eテストの設定が残っている可能性があるけど、e2eテスト間ってDBリセットされなかったっけ |
あーDBはリセットされるけどnestインスタンスは作り直されないのか |
その設計自体見直した方が良さそうだけど何か応急処置を考える必要がある |
TASUKETE |
通った |
マージするか |
* wip * Update ReactionService.ts * Update ApiCallService.ts * Update timeline.ts * Update GlobalModule.ts * Update GlobalModule.ts * Update NoteEntityService.ts * wip * wip * wip * Update ApPersonService.ts * wip * Update GlobalModule.ts * Update mock-resolver.ts * Update RoleService.ts * Update activitypub.ts * Update activitypub.ts * Update activitypub.ts * Update activitypub.ts * Update activitypub.ts * clean up * Update utils.ts * Update UtilityService.ts * Revert "Update utils.ts" This reverts commit a27d4be. * Revert "Update UtilityService.ts" This reverts commit e5fd9e0. * vuwa- * Revert "vuwa-" This reverts commit 0c3bd12. * Update entry.ts * Update entry.ts * Update entry.ts * Update entry.ts * Update jest.setup.ts
What
#14578
Why
See #14578
Additional info (optional)
Checklist