バージョン選択

フォーラム

メニュー

オンライン状況

37 人のユーザが現在オンラインです。 (31 人のユーザが フォーラム を参照しています。)
登録ユーザ: 1
ゲスト: 36
fb88luxury もっと...

サイト内検索

質問 > その他 > Git master へのマージに関して

その他

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
seasoft
投稿日時: 2024/1/14 12:33
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7367
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
居住地:
投稿: 2313
Re: Git master へのマージに関して
イーシーキューブ社の社員さんがマージ権限を持っていて、 Approve のついた PR を手動で順次マージするという流れです。

最近は人手不足でマージの時間がなかなか確保できない模様です。
こちらでも何かできることが無いか考えてみます

また、ユニットテストやE2Eテストの無いPRは確認に時間がかかるため、できるかぎりユニットテストやE2Eテストを作成していただけるとスムーズかと思います。

ユニットテストやE2Eテストの作成が難しいものは、スクリーンショットがあると確認がスムーズです。
seasoft
投稿日時: 2024/1/15 19:59
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7367
Re: Git master へのマージに関して
多分、2.17 の看板を背負っている部分は、LTS として運用せざるを得ないと思いますが、weekly についてはもっと積極的に吸収して良いのではないかと感じます。

https://github.com/EC-CUBE/ec-cube2/issues/681 にも絡むと思いますが、weekly を 2.17.x としてリリースする見込みか 2.18 向けかも明らかにするのが良さそうにも思います。


テストにつきましては検討したいと思いますが、たしか長期間放置されている PR の中には、テストやスクリーンショットとも違う (既存のテストをパスすることが正である) 性質のものもあった気がします。

その辺り、個別の PR には、マージ担当者からフィードバックが欲しいですね。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

red
投稿日時: 2024/1/15 20:43
対応状況: −−−
登録日: 2010/2/15
居住地: 東京都
投稿: 1570
Re: Git master へのマージに関して
weekly の延長が 2.17.x としてリリースする
が基本方針です

リリースにはイーシーキューブ社の手が回らない(オーナーズストアとかリリースとか)ので、開発者側から出来る限界でweeklyを出しているだけです。

マージ担当者さんは修正内容を判断できる状態ではないので、そういったことを求めるのも無駄な気がします

nanasessさんがさっとみてOKだせる状態でプルリクするのが親切です


----------------
EC-CUBEのカスタマイズ、トラブル解決承ります
お気軽にお問い合わせ下さい
https://www.ec-cube.net/integrate/partner/partner.php?partner_id=690

seasoft
投稿日時: 2024/1/15 21:27
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7367
Re: Git master へのマージに関して
引用:

多分、2.17 の看板を背負っている部分は、LTS として運用せざるを得ないと思いますが、weekly についてはもっと積極的に吸収して良いのではないかと感じます。

https://github.com/EC-CUBE/ec-cube2/issues/681 にも絡むと思いますが、weekly を 2.17.x としてリリースする見込みか 2.18 向けかも明らかにするのが良さそうにも思います。

パッチリリース (2.17.2-p1, 2.17.2-p2) と勘違いしてました。

2.17.2 は、eccube2-weekly-20211109 からリリースされてました。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

nanasess
投稿日時: 2024/1/17 17:15
対応状況: −−−
登録日: 2006/9/9
居住地:
投稿: 2313
Re: Git master へのマージに関して
イーシーキューブの中の人と相談しまして、今後、2系のリポジトリに関しては自動マージを導入する方針とのことです。

とり急ぎ、 PHPStan のエラーの原因となっていた、 html_checkboxes_ex, html_radios_ex 廃止のPRをマージしてもらいました。
適宜、テストが落ちている過去のPRも、最新の master を取り込んでいただいて、テストが通っている状態にしていただければ、自動マージ導入後に自動的にマージされる見込みです
seasoft
投稿日時: 2024/1/27 16:50
対応状況: 解決済
登録日: 2008/6/4
居住地:
投稿: 7367
Re: Git master へのマージに関して
ご調整いただきありがとうございます。

無難な PR から、最新 master をマージしようと思います。

この機にローカルでも E2E テストを実行できるように用意しようと思いますが、構成で不明点がありますので、別スレで質問させていただくかもしれません。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


EC-CUBE公式 Amazon Payプラグイン

統計情報

総メンバー数は88,801名です
総投稿数は109,972件です

投稿数ランキング

1
seasoft
7367
2
468
3217
3
AMUAMU
2712
4
nanasess
2313
5
umebius
2085
6
yuh
1819
7
h_tanaka
1644
8
red
1570
9
mcontact
1289
10
tsuji
958
11
fukap
907
12
shutta
835
13
tao_s
799
14 ramrun 789
15 karin 689
16 sumida 641
17
homan
633
18 DELIGHT 572
19
patapata
502
20
flealog
485


ネットショップの壺

EC-CUBEインテグレートパートナー

Copyright© EC-CUBE CO.,LTD. All Rights Reserved.