質問 > その他 > Git master へのマージに関して |
その他
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
seasoft |
投稿日時: 2024/1/14 12:33
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7369 |
Git master へのマージに関して EC-CUBE 2.17 へのプルリクを複数行っていますが、一向に master へ取り込まれず、新たな課題に取り組みにくいです。
https://github.com/EC-CUBE/ec-cube2/pulls/seasoftjapan 3年間以上マージされないものもあり、困惑しています。 マージの基準などを把握できていないのですが、誰かが手動で行っているのでしょうか? それとも、一定の基準で自動化しているのでしょうか? |
nanasess |
投稿日時: 2024/1/14 23:45
対応状況: −−−
|
神 登録日: 2006/9/9 居住地: 投稿: 2325 |
Re: Git master へのマージに関して イーシーキューブ社の社員さんがマージ権限を持っていて、 Approve のついた PR を手動で順次マージするという流れです。
最近は人手不足でマージの時間がなかなか確保できない模様です。 こちらでも何かできることが無いか考えてみます また、ユニットテストやE2Eテストの無いPRは確認に時間がかかるため、できるかぎりユニットテストやE2Eテストを作成していただけるとスムーズかと思います。 ユニットテストやE2Eテストの作成が難しいものは、スクリーンショットがあると確認がスムーズです。 |
seasoft |
投稿日時: 2024/1/15 19:59
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7369 |
Re: Git master へのマージに関して テストにつきましては検討したいと思いますが、たしか長期間放置されている PR の中には、テストやスクリーンショットとも違う (既存のテストをパスすることが正である) 性質のものもあった気がします。 その辺り、個別の PR には、マージ担当者からフィードバックが欲しいですね。
|
red |
投稿日時: 2024/1/15 20:43
対応状況: −−−
|
神 登録日: 2010/2/15 居住地: 東京都 投稿: 1574 |
Re: Git master へのマージに関して weekly の延長が 2.17.x としてリリースする
が基本方針です リリースにはイーシーキューブ社の手が回らない(オーナーズストアとかリリースとか)ので、開発者側から出来る限界でweeklyを出しているだけです。 マージ担当者さんは修正内容を判断できる状態ではないので、そういったことを求めるのも無駄な気がします nanasessさんがさっとみてOKだせる状態でプルリクするのが親切です
|
seasoft |
投稿日時: 2024/1/15 21:27
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7369 |
Re: Git master へのマージに関して 引用:
パッチリリース (2.17.2-p1, 2.17.2-p2) と勘違いしてました。 2.17.2 は、eccube2-weekly-20211109 からリリースされてました。
|
nanasess |
投稿日時: 2024/1/17 17:15
対応状況: −−−
|
神 登録日: 2006/9/9 居住地: 投稿: 2325 |
Re: Git master へのマージに関して イーシーキューブの中の人と相談しまして、今後、2系のリポジトリに関しては自動マージを導入する方針とのことです。
とり急ぎ、 PHPStan のエラーの原因となっていた、 html_checkboxes_ex, html_radios_ex 廃止のPRをマージしてもらいました。 適宜、テストが落ちている過去のPRも、最新の master を取り込んでいただいて、テストが通っている状態にしていただければ、自動マージ導入後に自動的にマージされる見込みです |
seasoft |
投稿日時: 2024/1/27 16:50
対応状況: 解決済
|
神 登録日: 2008/6/4 居住地: 投稿: 7369 |
Re: Git master へのマージに関して ご調整いただきありがとうございます。
無難な PR から、最新 master をマージしようと思います。 この機にローカルでも E2E テストを実行できるように用意しようと思いますが、構成で不明点がありますので、別スレで質問させていただくかもしれません。
|
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |