git push -f が更に安全になる --force-if-includes - id:onk のはてなブログ

歴史改変、してますか？ 私は歴史改変が大好きで、毎日 rebase しています。なので割と毎日 git push -f することになっています。 口で -f と言っても、実際には --force-with-lease --force-if-includes をしているので、これらのオプションのご紹介。 この記事は はてなエンジニア Advent Calendar 2022 の 18 日目です。昨日は id:rokoucha さんで 壊れたデータベースとの向きあいかた - rokoucha でした。 qiita.com -f の危険性 ...--F--G--H <-- main という状態で push した後、H をコミットし直したとしよう。 ...--F--G--H' <-- main \ H <-- origin/main このまま H' (main) を origin/main に p

[onk]

書きました。CI が積んだコミットがよく闇に葬られる。あと稀に同じブランチで共同作業することになるのと、ポカヨケが目的です。

[tagomoris]

mainに積まれた歴史を書き換えるのにあれこれ頑張るより、mainに積まれたらもう書き換えない、というポリシーの方が好きだな。

[igagurimk2]

単なる例示でしかないブランチ名mainに引っ張られて「mainにforce pushなんてしない」なんて言っている人がいるので、feat-x とかの方が良いのだろうなぁ。

[yujiorama]

force-with-leaseはgitconfigで初期値にできないからaliasにすると安全そう。push.default=simple push.useForceIfIncludes=yesは設定して、どうしてもとなったら--no-force-if-includesとするのもよさそう https://git-scm.com/docs/git-push

[dekokun]

自分一人で作業してるブランチのはずが…というポカをしたことのある私としてはですね、これは必須ですね。

[tettekete37564]

これ自分しか使ってないリモートリポが前提だよね？共同作業者がいるならやめましょ？どうしてもという時はrevertコミットして履歴に残しましょ？

[kazuomabuo]

1回やった。手動リカバリして、対応してくれた方に迷惑かけた。。。

[gfx]

知らなかった。よさそう。まあpush -fは自分ひとりしかコミットしないブランチでしかしないけど…。

[snowcrush]

便利そうだけどそもそもチーム開発時に他人と共有しているブランチにforce pushしないからあまり使いどきはないかな…。

[sue445]

git基礎勉強会懐かしい。もう9年半経ってるの...

[k2784]

#読んだ

[hajimepg]

GitHubなどの共用リポジトリに一度でもプッシュしたらもう書き換えないスタイルでやってる。どうしても直したいなら、main(master)から別ブランチを生やして、元のブランチとは歴史を共有しない形にする

[rochefort]

--force-if-includes デフォルトの挙動これにしといて欲しい気もする

[hamaco]

--force-if-includes 知らなかった。追加した。

[curin020304]

Git -f について

[mas-higa]

こうして I は守られたわけだけど、H' はどうやって捩じ込んでるの?

[hshimoyama]

--force-with-lease しか使ってなかった。--force-if-includes も添えるようにしよう

[nilab]

git push -f が更に安全になる --force-if-includes - id:onk のはてなブログ

[odz]

まず、共有しているブランチにforce pushすな。

[bellonieta]

--forceの形骸化

[nakag0711]

オプションてんこ盛りのgitコマンド。こういうのGUIのメニューでやりたいわ

[kobito19]

"「push -f」先がmainか他ブランチかどうかは極めて重要な問題" 自分の認知してないコミットが消えるのが問題なのは別に main かどうかに依存せんやろ / alias 設定は自分もこんな感じ https://qiita.com/ototonari/items/fd67801a2812f33ace1a

[arx0balest]

脳死force-with-leaseを脳死force-if-includesに変えりゃええんか？

[masutaka26]

force push は避けてはいるけど、とりあえず $ git config --global alias.push 'push --force-with-lease' && git config --global push.useForceIfIncludes yes しておいた

[Windymelt]

magitで使えるようにならないかな〜

[tocguitar1]

これは知らなかった。alias登録せねば。

[mitaro]

あとでコンフィグで設定できるか調べよう。/すでにブコメされてた。感謝。/設定で安全側に倒せるものは倒しておきたい。

[tookikimie]

原則自分の作業ブランチか、周りに確認してからしかやらんけど、このオプション付けててタイポ無かったことにしたり変な改行とか除いたりとかできるんか？

[seiunsky]

めちゃ丁寧に解説されていて良い……

[masa8aurum]

mainブランチ云々のブコメがついてるのは結局、例示が悪いという話なので、今からでもfeature-aとかに直すほうがいいと思う（それかせめてdevelopにするとか）

[koo-sokzeshky]

“私は歴史改変が大好きで、”

[bouzuya]

より安全になるので良さそう。ぼくの場合は push -f する先のブランチに他人のコミットが含まれない運用になのと、それが含まれ得るブランチ（例えば main や master ）は protect されてるので -f でも実害はなさそう。

[gachapining]

これ面白い。"オススメは .git の中で git init して、 git fetch や git push したときに何が変わるのかを diff の形で眺めてみることです"

[NetPenguin]

実施していきたい

[Fluss_kawa]

mainに積んだ上で他人がもう新しいの入れちゃったら素直に訂正したい内容を新たに作って積むなぁ。