#1261 の修正を 2.4.1 beta1 にマージする

[beru]

(必須) PR の目的

#1261 の修正を 2.4.1 beta1 にマージする

(必須) カテゴリ

• 不具合修正

(なければ省略可) 関連 issue, PR

#1249, #1261

[beru]

release/v2.4.1-beta1 ブランチが作成された後に master に merge された commit をrelease/v2.4.1-beta1 ブランチにも取り込む必要がありますが、これっていちいちPR作成する必要ってあるんでしょうか？

単に担当者が cherry-pick して push すれば良いような気もするんですが…。なんかいちいち確認のプロセスを経由してると身動きが遅くならないですかね。

[AppVeyorBot]

✅ Build sakura 1.0.2786 completed (commit 8e7f4dd2b0 by @berryzplus)

[berryzplus]

release/v2.4.1-beta1 ブランチが作成された後に master に merge された commit をrelease/v2.4.1-beta1 ブランチにも取り込む必要がありますが、これっていちいちPR作成する必要ってあるんでしょうか？

その必要は、ないような気がしますが、PRにせざるを得ないんではないかと思います。

単に担当者が cherry-pick して push すれば良いような気もするんですが…。なんかいちいち確認のプロセスを経由してると身動きが遅くならないですかね。

う～ん・・・やり方、どうするのがいいんだろう？

こいつ含む一連のマージPR(#1279, #1280, #1281)に対して異論はありませんが、「やり方」っちゅう話になると「もっと分かりやすい別なやり方もあるんじゃね？」と思います。

というわけで、ちょっと考えたいっす:smile:

[berryzplus]

詳しい人、意見求ム・・・

そもそもこのPRのマージ先って、 #1249 で新たに作ろうとしているブランチなはずで、まだ存在していないんではないか？とか思ったりもするわけで。

[beru]

そもそもこのPRのマージ先って、 #1249 で新たに作ろうとしているブランチなはずで、まだ存在していないんではないか？とか思ったりもするわけで。

存在していない事は無いですよ。存在していなかったらそもそも merge 先の branch に github の web UI で指定出来ないと思います。#1257 が merge されたりもしています。

ここのリポジトリに release/v2.4.1-beta1 ブランチを切ったのが誰かが把握してませんが、多分 @KENCHjp さんじゃないですかね？自分が入れてしまった不具合(Issue #1233) が結構クリティカルなので PR #1241 がマージされた後になる早でリリースするつもりで release/v2.4.1-beta1 ブランチを作成して PR #1249 を作成したんじゃないかと推測します。

[m-tmatma]

ブランチを最初に作るのは直pushで、作成できますが、
いったん作ったら、作ったブランチは直pushは保護されるという動きのはずです

[beru]

https://github.com/sakura-editor/sakura/tree/release/v2.4.1-beta2
作成しました。この issue は close します。

[berryzplus]

なんかを勘違いしてたのがぼんやり分かったような・・・。
とりあえず今回はbeta1が欠番になってbeta2作成作業が始まる感じですね。