Skip to content

Conversation

@h-east
Copy link
Member

@h-east h-east commented Nov 17, 2024

No description provided.

@h-east h-east changed the title Update develop Update develop.{txt,jax} Nov 17, 2024
@h-east
Copy link
Member Author

h-east commented Nov 17, 2024

forkリポジトリに間違ってスタイル変更の help.jax をcommit&pushしちゃって後から revert したけど、それ以降のPRに変なcommit付いてしまうので、help.jax のcommit&revert の分を rebase でdropしたんですけど、、、(長い)
このPR、大丈夫ですかね!?

@h-east
Copy link
Member Author

h-east commented Nov 17, 2024

このPRを先にmergeしちゃうと、他のPR変なことになる?

@mityu
Copy link
Member

mityu commented Nov 17, 2024

forkリポジトリに間違ってスタイル変更の help.jax をcommit&pushしちゃって

これブランチは master ですかね?
fork 先リポジトリの master ブランチの履歴をちょっと眺めたんですが、何件か同じ内容のコミットがある(例えばこれこれ)ので、help.jax をcommit&pushしてしまったときに色々他のコミットも混ざり込んでしまった気配がします。

多分 master については h-east@7a2e8a1 からこれも含めた先のコミットを落とすと、以降の PR でも「それ以降のPRに変なcommit付いてしまう」というのが良い感じになおるんじゃないかと思います。
この PR についてはすでにブランチが切られてるんで同様の対処がもう一度必要ですけど。(cherry-pick した方が楽?かも?)

このPRを先にmergeしちゃうと、他のPR変なことになる?

ちょっと git とか GitHub とかの細かい挙動がよくわかってないのでわかりません...が、この PR の差分表示タブの内容を見る感じ無事そうな気はしますね。

@h-east h-east closed this Nov 17, 2024
@h-east h-east deleted the update-develop branch November 17, 2024 16:02
@h-east
Copy link
Member Author

h-east commented Nov 17, 2024

@mityu ありがとうございます。
まともな Update develop PR 作成できました。 #1834

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.

2 participants