smngs / da-bot

Discord bot for 'da'-server!
1 stars 0 forks source link

[update] using `aiohttp` to request for speak-chat, aichat #35

Closed smngs closed 1 year ago

smngs commented 1 year ago

Issue

変更の概要

変更の理由(なぜこの変更をするのか)

その他

BREAKING CHANGE

smngs commented 1 year ago
12ev09 commented 1 year ago

まだレビュー途中ですが、squash mergeでマージしましょう

smngs commented 1 year ago

まだレビュー途中ですが、squash mergeでマージしましょう

個人的には Squash Merge の利点があんまり分かってない(せっかくコミットである程度の粒度にまとめたのに,それを一つのコミットにまとめる理由が分からん)ので,積極的な理由があるなら教えてください.

12ev09 commented 1 year ago

個人的には Squash Merge の利点があんまり分かってない(せっかくコミットである程度の粒度にまとめたのに,それを一つのコミットにまとめる理由が分からん)ので,積極的な理由があるなら教えてください.

(PRの粒度が小さいという前提で)コミットが多い場合は、不具合の修正や、レビュー後の訂正が多いと考えられるのでコミットが全て残ってることのメリットが小さいと思う。(mainを見返したときのあるコミットが、レビュー後の修正のコミットだったらあまり参考にならない。)

と考えてるんだけど、今回の場合はPRの粒度が大きめだから多少のコミット数は良いか思うので今回はしなくて良い。