Skip to content

Git作業のヒント

Soichiro Miki edited this page Aug 27, 2023 · 4 revisions

大量のコミットをプッシュしない

  • 「20行目まで翻訳」「40行目まで翻訳」…
  • 「翻訳開始」「途中まで翻訳」「仕上げ」「微調整」「追加で修正」…

のような作業中コミットは、ローカル作業でやるぶんには構いませんが人に見せるものでもマージして永久に記録に残すべきものでもありませんので、なるべくプッシュは控えめに行い、レビュー前にコミットを git rebase でまとめてください(やり方は「コミット rebase まとめる」などで検索してください)。ドラフト PR にこのようなコミットを大量にプッシュしてしまった場合は、"ready for review" にする前に rebase + force-push してください。

小さなコミットをあまりに頻繁に(例えば 10 分ごとに)プッシュしすぎないようにしてください。プッシュごとにGitHub Actions や Vercel での再ビルド・再デプロイやバンドルの解析が毎回 10 分ほど走り、そこそこの負荷になります。プレビューも半永久的に残ります。Meta が支払ってくれているとは思いますが一応有料です。

image

その日の作業の終わりに 1 回だけプッシュする、くらいならまったく問題ありません。

GitHub 上でのサジェスチョンのバッチ適用

レビュー時にかなり多くのコメントが付く場合があります。毎サジェスチョンごとに GitHub 上でコミットを行うと、上記と同様に自動テスト・解析・プレビュー作成が毎コミットごとに発生し、かなりの負荷になります。代わりに、修正のサジェスチョンはバッチにまとめて 1 コミットで適用するようにしてください。以下の "Files changed" タブからファイル差分を表示すると、"Add suggestion to batch" というボタンがあるはずですので、それで作業するようお願いします。

image