Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[project-library-manage] mainをマージ #1389

Merged
merged 3 commits into from
Jul 12, 2023
Merged

Conversation

y-chan
Copy link
Member

@y-chan y-chan commented Jul 12, 2023

内容

題のとおりです、mainに取り込まれたキャラボイス試聴コンポーネントやbrand型のUUID矯正を解除する2つの変更をrebaseでマージします。

関連 Issue

スクリーンショット・動画など

その他

このPRをマージするために、一時的にPull Requestマージ設定のrebaseをonにします

@y-chan y-chan requested a review from a team as a code owner July 12, 2023 14:03
@y-chan y-chan requested review from Hiroshiba and removed request for a team July 12, 2023 14:03
@Hiroshiba
Copy link
Member

rebaseします!

rebaseで良いかのDiscordでの議論メモ

ヒホ
rebaseマージって、そのブランチにマージされたPRとかはちゃんと追っかけれるんですかね・・・?

y-chan — 今日 23:08
多分追っかけてくれるかなと...!
今回の場合は枝はわかれずまんまmainと同じ枝の上に乗るはずです...! 

ヒホ — 今日 23:09
うーむ commi idは変わるはずだよな~とか思ってます
あとはコンフリクト生じたときは記録残すためにmergeがあったほうが良さそうだから、生じないときだけrebase・・・?
あまり深く考えてなかったけど、意外とrebase微妙かもしれない

y-chan — 今日 23:12
rebaseでコンフリクトがでればコミットの作り直しになると思うので、commit idは変わると思いますが、それ以外はcommitは全く変化しないはず...
今回の場合はrebaseしてもmainと同じ枝の上に乗るだけなので、まあなにか影響が出ることはないかなと思いますね
と思ったけど、この場合prコミットが残らないってことになる...?どうやってPRがマージされたって判断するんだろう

ヒホ — 今日 23:14
そこはまあマージボタンが押されたかどうかじゃないですかね~
PR出たブランチをlocalでmainにマージしてpushしてもマージされたことにはならないだろうし
rebaseはcommit idは変わるはず。じゃないと全く同じcommit idが2つ以上現れるので衝突しちゃう。
PRがcommit idに紐付けられてた場合はrebaseで切れそう。
commit履歴→PRへはマージ時にPR番号がcommit messageに記録されるから切れないけど、PR→commit履歴は切れそう感
なのでたぶん良くないんだけど、ちょっとやってみますか!w

@Hiroshiba Hiroshiba merged commit f3fd114 into project-library-manage Jul 12, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants