-
-
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
Fix: <link rel="alternate">
を追って照会するのはOKレスポンスが返却された場合のみに
#14627
Fix: <link rel="alternate">
を追って照会するのはOKレスポンスが返却された場合のみに
#14627
Conversation
このPRによるapi.jsonの差分 |
メモ: sourceはシンプルに「ダウンしてる時に追うのはよくない」と言っている |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
実質2xxのチェック追加のみ
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14627 +/- ##
===========================================
- Coverage 39.67% 39.57% -0.10%
===========================================
Files 1542 1542
Lines 192781 192785 +4
Branches 2475 3540 +1065
===========================================
- Hits 76492 76302 -190
- Misses 115726 115886 +160
- Partials 563 597 +34 ☔ View full report in Codecov by Sentry. |
🙏🏻 |
…v#14627) * Only accept HTML `<link rel="alternate">` on success (cherry picked from commit 6ea48be) * Use `res.ok` instead of 200-299 (cherry picked from commit b667a68) * jsdomに戻す * Update Changelog * Revert "jsdomに戻す" This reverts commit c036036. * 🎨 --------- Co-authored-by: Julia Johannesen <[email protected]>
…v#14627) cherry picked from commit dd124a8 Co-authored-by: Julia Johannesen <[email protected]> Co-authored-by: かっこかり <[email protected]>
What
Why
Additional info (optional)
Checklist