開発の手続きについて
Issue #96
new
現状、CardWirthPy Rebootの開発は次のような形で進んでいます。
k4nagatsukiの場合
- 自分のリポジトリでcommitする
- そのままbitbucket上のCWPyのリポジトリへpushする
k4nagatsuki以外の貢献者の方
- 自分のリポジトリでcommitする
- bitbucket上の自分のリポジトリへpushする
- Pull Requestし、k4nagatsukiがレビュー
- マージ
今のところアクティブな開発者の中でk4nagatsukiのみがコードの全体を把握しているという状況があり、そのためcommitorとreviewerがk4nagatsuki以外にいないためにこのような現状となっています。
しかし、これには問題があります。k4nagatsukiがcommitしたコードをレビューする人がいません。これは将来的に改善していかないとなりません。具体的には、私も貢献者の方と同様のフローでコードをpull requestしなければいけません。さもないと#94のようなポカミスが頻発する事になります。
まずは第一歩として、私自身の開発用のリポジトリをbitbucket上に作りました。結局は私自身がメインストリームのリポジトリに取り込むことになるにせよ、今後は私用のリポジトリからpull requestする形で開発を進める事にします。
後は私のコードをレビューする方がいない、という問題ですが、「私もpull requestのレビュー担当者になってもいいよ」という方は是非とも名乗り出てほしいと思います。更新の内容が分からない場合はpull requestした人に「これはどういう更新なのか？」と質問して説明させればよいのです。それによって知識も増すはずです。
そうして「もうCWPyのコードはだいたい把握しているから、commitorとしても大丈夫」というように思えるようになったら、それもまた名乗り出ていただければ、メインストリームのリポジトリへの書き込み権限を差し上げて、私と共にpull requestを取り込む役割を担っていただけるかと思います。
Comments (6)
-
reporter
-
reviewer ……うぅーん、ごめんなさい。今の自分ではまだ、荷が勝っております……。最初の頃より知識が増えたとはいえ、自分の実感としてはまだ、食べやすいところだけ齧ってる状態です。これまで出された他の方の修正内容を見ても、ジャッジするどころか内容を半分も理解できなかったり。
申し訳ありませんが、自分はもうしばらく未参加のままでいきたいと思います。
ただ今回の話、どこを目指したいかはよく解ります。それに自分も何かしら貢献できたら。β直前直後の修正ラッシュが落ち着いてきたら、reviewer ではなくても commit に関する質問を飛ばしたりし始めるかもしれません。そのときは、ご面倒かと思いますが、お相手いただければ幸いです。
（……こりゃ「もっと詳しい人がいるし」なんて言ってる場合じゃないぞ。当事者意識を高めないと）
-
reporter
了解です。理解できる部分だけやっていくというスタイルも大いにありだと思います。そのほうがずっと長続きしそうというのもありますし。
商売でやっているわけでもなし、このソフトの開発も優先度は人それぞれなので、自分の生活の中に自分なりに位置づけて気楽に参加していただければと思います。
-
reporter
あ、質問とかはいつでもどうぞ。できるだけ回答したいと思います。
-
見ました。
今現在 k4nagatsuki さんにかかる負担が大きい問題は承知しておりますが、一つの修正にも割と数日かかっちゃうような自分がコードレビューしても問題です。自分としては「気がついたから修正する」今のスタイルがちょうどいい感じであり、なおかつ精一杯でもあります。
現状では各々がコミットした最新版を各々がテストプレイしてて、これがすごく間接的なコードレビューになっていると思えば、それはそれでいいかなと思ってます。あくまでも、これは個人の意見ですが。
ぼ、某スレの積み感想も貯まってるし…（及び腰）
-
reporter
了解しました。それぞれのペースで参加していただければ。
- Log in to comment
本音を言うと今の貢献者の方にはぜひともすぐreviewerになっていただきたいのですが、当然レビューするには相応の負担が発生するので、現状のままでももちろん結構です。
現状は私の独裁なので、徐々にもっと民主的な開発に移行していければいいなと思ってます。
しかし誰かreviewerになったとして、ここでPullRequestが発生した通知を受け取るにはリポジトリのウォッチ機能を使えばいいのかな？ よくわからん……。