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

v2.4.1 beta1 リリース #1249

Closed
wants to merge 1 commit into from
Closed

v2.4.1 beta1 リリース #1249

wants to merge 1 commit into from

Conversation

KENCHjp
Copy link
Member

@KENCHjp KENCHjp commented Apr 29, 2020

PR の目的

v2.4.0 リリース後に発生したクリティカルな不具合の修正。
本PR問題なければタグうちしてpre-release作成する。
beta1リリース後1week程検証し、問題なければbeta1を元にリリースブランチを作成してリリースする。問題あればmasterよりbeta2をブランチして再テスト、以降繰り返す。

カテゴリ

  • 不具合修正
  • リリース

PR の背景

PR のメリット

PR のデメリット (トレードオフとかあれば)

PR の影響範囲

関連チケット

#1241
#1238

参考資料

@AppVeyorBot
Copy link

Build sakura 1.0.2738 completed (commit 77292ab73a by @KENCHjp)

@beru
Copy link
Contributor

beru commented Apr 29, 2020

どなたか berryzplus さんの環境で起きた #1246 (comment) の現象について確認してもらえないでしょうか?他の方の環境でも同様の現象が出る場合は revert する PR を作成しようと思います。

@berryzplus
Copy link
Contributor

検証中・・・?

ウチの端末で起きてるマウスドラッグ時の描画がおかしくなる問題は、実害なさそうなのでスルーでいい気がします。

@beru
Copy link
Contributor

beru commented May 4, 2020

#1246 (comment) の現象ですが、https://discordapp.com/channels/457826864879370250/458844734690033664/706754940155985960
の報告によると問題が有るみたいですね。という事で revert するPRを作成します。

@beru
Copy link
Contributor

beru commented May 4, 2020

#1265 を作成しました。

Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このPRの変更内容は問題無いと思いますが、 release/v2.4.1-beta1 に含めたい内容は他にもあるので別途PRを作成します。

@KENCHjp
Copy link
Member Author

KENCHjp commented May 5, 2020

おお、これはマージ、まっててレビュー通ればそのままリリース作業入っていいのかな?

ヘルプのマージとか入っていますが、どなたか流れを支援プリーズ。

@beru
Copy link
Contributor

beru commented May 5, 2020

ヘルプのマージとか入っていますが、どなたか流れを支援プリーズ。

まだ修正が出揃っていないのでもう少々お待ちください。

v2.4.0 リリースには色々バグが含まれている事が判明したのでその修正を行ったリリースをなるべく早く出す必要がありますが、まだ修正がきちんと出来ていない状況です。

あと v2.4.0 リリース後に merge されたPR #1246 に問題が有る事が判明したので #1265 で revert しました。そのコミットを release/v2.4.1-beta1 ブランチに含めたいです。

@KENCHjp
Copy link
Member Author

KENCHjp commented May 7, 2020

個別にマージPRが手間ならいっそBeta1をDeadにしてmasterから改めてBeta2作りましょうか。

@KENCHjp
Copy link
Member Author

KENCHjp commented May 7, 2020

不具合の内容をみてないのであれですが、2.4.1は選択文字列の字数間違いを早めにリリースしたい意図があるので、仮に対応中の不具合がデグレードであっても致命傷でなければ、次リリースでもいいかなと思っています。
時間と内容との兼ね合いですがいいころ合いでコメントいただければ良しなに作業します。

@berryzplus
Copy link
Contributor

う~む。こうしたい・・・(肌色がこのPR、緑色がピックアップで取り込みたいPR、赤色が統合されたブランチ)

merge-plan-20200507

これって実は「このままpushしたらOK」とか、そういう話?(よく分かっとらん

@beru
Copy link
Contributor

beru commented May 7, 2020

@berryzplus さん

う~む。こうしたい・・・(肌色がこのPR、緑色がピックアップで取り込みたいPR、赤色が統合されたブランチ)

出来るなら PR #1255 のコミットも取り込んだ方が良い気はします。こちらで取り込みPRを作成しておきますか?

これって実は「このままpushしたらOK」とか、そういう話?(よく分かっとらん

push 出来る権限があるなら、他メンバーに告知すれば混乱は生じないのでOKだと思います。

masterブランチに直接 push とかはそこそこ多い人数が関わってるプロジェクトなので問題あると思いますが、release用のブランチであればメンバーに周知すれば問題無い気がします。

もしそうされるなら自分が作成したPR群 #1279, #1280, #1281 は close します。

ただもし不安ならberryzplusさん本人以外のメンバーにもチェックを行ってもらうのが良いと思います。が、その場合はPRを作成するやり方でもいっしょかもしれません。自分はひとつひとつ個別にPRを作成しましたが、一気にまとめて対応したPRでも良い気はします。

色々なやり方が考えられますが、まぁどの方法でも大体問題無いんじゃないかなと思います。あと今 master に入っていてまだ release/v2.4.1-beta1 に取り込まれていないコミットは無くは無いと思いますが、今色々調べて全部含める努力をしないでも、また後で v2.4.1-beta2 を作る時には master から派生するので大丈夫だろう、という考えもあります。

あと v2.4.1-beta1 リリース前には、コミットの取捨選択による思わぬ不整合によって問題が発生していないかを少しは動作確認する必要がありますね。

@beru
Copy link
Contributor

beru commented May 7, 2020

@KENCHjp さん

個別にマージPRが手間ならいっそBeta1をDeadにしてmasterから改めてBeta2作りましょうか。

Beta1 を廃版にしないでも手元の release/v2.4.1-beta1 ブランチを削除して再度 master から release/v2.4.1-beta1 ブランチ作成しなおして force push する方法でも問題は無いと思いますよ。ただその場合は事前にメンバーに周知した方が良いですね。release/v2.4.1-beta1 ブランチを過去に pull した人は force pull する必要が出てくると思うので。

@m-tmatma
Copy link
Member

m-tmatma commented May 7, 2020

個別に pr するメリットは、レビューするのもそうですが、
pr という形で記録が残るので、後でどんな修正を入れたか、後で確認しやすいことです。

@m-tmatma
Copy link
Member

m-tmatma commented May 7, 2020

releaseブランチは protect 設定されているので、直 push はできないようにしてます。

単純に beta2 作るのが、簡単でいいと思います

@beru
Copy link
Contributor

beru commented May 7, 2020

なるほど、それでは誰かが release/v2.4.1-beta2 ブランチを作成するなら #1249, #1279, #1280, #1281 は close ですね。

@beru
Copy link
Contributor

beru commented May 7, 2020

@sakura-editor/sakura-developers
release/v2.4.1-beta2 ブランチを作成します。

@beru
Copy link
Contributor

beru commented May 7, 2020

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

@beru beru closed this May 7, 2020
@KENCHjp
Copy link
Member Author

KENCHjp commented May 8, 2020

はい、本件了解です。beta1はDeadで(タグ打たない)、beta2作りましょう。

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.

5 participants