misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
9.95k stars 1.35k forks source link

fix(backend): リバーシの設定変更が反映されないのを修正 #14404

Closed kakkokari-gtyih closed 1 month ago

kakkokari-gtyih commented 2 months ago

What

Why

Fix #14397

Additional info (optional)

Checklist

github-actions[bot] commented 2 months ago

このPRによるapi.jsonの差分

差分はこちら ```diff ```

Get diff files from Workflow Page

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 24.24242% with 25 lines in your changes missing coverage. Please review.

Project coverage is 41.74%. Comparing base (41936c1) to head (7e7e999). Report is 5 commits behind head on develop.

Files Patch % Lines
packages/backend/src/core/ReversiService.ts 25.00% 21 Missing :warning:
...end/src/server/api/stream/channels/reversi-game.ts 20.00% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #14404 +/- ## =========================================== + Coverage 39.91% 41.74% +1.83% =========================================== Files 1545 1549 +4 Lines 190505 196580 +6075 Branches 2654 3589 +935 =========================================== + Hits 76046 82070 +6024 - Misses 113898 113949 +51 Partials 561 561 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

kakkokari-gtyih commented 2 months ago

@bindThisって基本入れたほうがいいのかしら

anatawa12 commented 2 months ago

@bindThis自身はinstance.someMethodって引数で渡すことが一切ないならなくても困らないはずだけど、今後起きることを考えるとつけとくと良いと認識してます。

javascriptではconst a = instance.someMethodってやったあとにa()って呼ぶとaの中でthisinstanceと違う値になって壊れることがよくあります。これの対策と思われます

kakkokari-gtyih commented 2 months ago

いれた

syuilo commented 1 month ago

🙏🏻