Closed ltlapy closed 1 week ago
Attention: Patch coverage is 0%
with 8 lines
in your changes missing coverage. Please review.
Project coverage is 65.22%. Comparing base (
379ce01
) to head (d2d660c
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
テストに明示的にあるし、なぜ +1 にするのかをしっかり考えたほうが良さそう。
また、互換性的にはマイグレーションで +1 したいきもする (1つを意図して0にしてたところが作れなくて混乱が発生しそう)
昨日軽く履歴を追いかけたけど、「テストは後付 (#10358) で、もともとの実装がそうなっていたのを書き起こしたから」以上のことは言えなさそう
>
マイグレーションはまぁそう
This fix #12279
1件ずれているのがバグなのか仕様なのかで対応が変わりそうです。
@syuilo 過去の経緯など、なにかご存じだったりしますか…?
覚えてないけどバグじゃないかしら
🙏
また、互換性的にはマイグレーションで +1 したいきもする
これは別issueで考えたい感じです?
少なくとも CHANGELOG 要ると思います
少なくとも CHANGELOG 要ると思います
→ #14037
What
Why
Additional info (optional)
/packages/backend/src/queue/processors/ImportUserListsProcessorService.ts
を参考Checklist