提案チケットの進め方 » 履歴 » バージョン 10
toshi_a 初音, 2019-11-26 21:34
1 | 1 | toshi_a 初音 | {{>toc}} |
---|---|---|---|
2 | |||
3 | h1. 進め方 |
||
4 | |||
5 | 6 | toshi_a 初音 | トラッカー「提案」は、[[ロール#報告者|報告者]]が、新機能の提案や、バグではない軽微な変更をパッチを送る場合に使われる。 |
6 | |||
7 | このトラッカーはパッチを必ず添付しなければならない。 *パッチの無い機能要望などは、mikutterでは受け付けていない* 。 |
||
8 | 1 | toshi_a 初音 | |
9 | 基本的な流れとしては、 |
||
10 | |||
11 | # [[ロール#モデレータ|モデレータ]]が、その提案をmikutterにマージすべきか判断する(分類待ち) |
||
12 | # マージすべきと判断した場合(パッチ適用待ち) |
||
13 | # 修正パッチを試用し、レビューする(レビュー待ち) |
||
14 | # 適切なブランチにmergeする(マージ待ち) |
||
15 | |||
16 | というふうになる。 |
||
17 | |||
18 | h2. ステータス「分類待ち」の場合 |
||
19 | |||
20 | この場合、[[ロール#モデレータ|モデレータ]]がその提案が妥当なものなのか判断し返信する。 |
||
21 | |||
22 | 3 | toshi_a 初音 | h3. パッチを取り込むべきではない場合 |
23 | |||
24 | 1 | toshi_a 初音 | 妥当でないならばその理由を説明し、ステータスを「却下」に変更する。 |
25 | |||
26 | 3 | toshi_a 初音 | h3. パッチを取り込むべきか判断できない場合 |
27 | 1 | toshi_a 初音 | |
28 | 3 | toshi_a 初音 | 議論をしてもacceptするかrejectするか判断できない場合、または議論の余地がなく、toshi_aがどうするか判断するしかないと考えられる場合、チケットを以下のように更新する。 |
29 | 1 | toshi_a 初音 | |
30 | 5 | toshi_a 初音 | * *ステータス*: 「toshi_aの判断待ち」 |
31 | 3 | toshi_a 初音 | |
32 | 4 | toshi_a 初音 | この場合、 *担当者を変更する必要はない* 。 |
33 | 3 | toshi_a 初音 | |
34 | このステータスへの遷移は、[[ロール#モデレータ|モデレータ]]の誰か一人でもtoshi_aが判断すべきと考えたら即座に行って良い。 |
||
35 | |||
36 | h3. 取り込むべきと判断できる場合 |
||
37 | |||
38 | 10 | toshi_a 初音 | [[ロール#モデレータ|モデレータ]]がそれを取り込むべきと判断すれば、チケットを以下のように更新する。 |
39 | |||
40 | - *ステータス*: 「パッチ適用待ち」 |
||
41 | 3 | toshi_a 初音 | |
42 | 1 | toshi_a 初音 | h2. ステータス「パッチ適用待ち」の場合 |
43 | |||
44 | このステータスの時にやるべきことは、単にそのパッチをコミットすることである。したがって、このステータスのチケットは[[ロール#コミッタ|コミッタ]]しか操作することができない。 |
||
45 | |||
46 | h3. パッチをコミットする |
||
47 | |||
48 | [[ロール#コミッタ|コミッタ]]は、「パッチ適用待ち」のコミットを見たら、いつでもこのパッチをコミットできる。 |
||
49 | |||
50 | 9 | Yuto Tokunaga | 詳細は [[#ブランチの起点について|ブランチの起点について]] に従う。 |
51 | 1 | toshi_a 初音 | |
52 | ただし、コミットする時にはgitコマンドの--authorオプションを使って、パッチを作った人をAuthorにすること。 |
||
53 | |||
54 | このブランチをPushしたら、チケットのステータスを「レビュー待ち」に変更する。 |
||
55 | |||
56 | この時、次のフィールドをそれぞれ以下のようにする。 |
||
57 | |||
58 | - *ステータス*: 「レビュー待ち」 |
||
59 | - *ブランチ名*: コミットしたブランチ名を入れる。 |
||
60 | - *担当者*: ([[ロール#修正者|修正者]]) |
||
61 | 3 | toshi_a 初音 | |
62 | h2. ステータス「toshi_aの判断待ち」の場合 |
||
63 | |||
64 | このステータスの場合、toshi_aが「パッチ適用待ち」または「却下」にする。 |
||
65 | |||
66 | このステータスになっていても、他のユーザがこのチケットについて意見を注記に書いて良い。 |
||
67 | 1 | toshi_a 初音 | |
68 | h2. ステータス「レビュー待ち」の場合 |
||
69 | |||
70 | このステータスになる時、必ずカスタムフィールド「ブランチ名」に、修正がコミットされたブランチが書いてある。 |
||
71 | |||
72 | h3. チケットの担当者が設定されてる場合 |
||
73 | |||
74 | この場合は、設定された担当者がそのブランチをチェックアウトして動作確認を行う。 |
||
75 | |||
76 | 結果、このチケットで提案している機能が正しく実装できていることが確認できたら、ステータスを「マージ待ち」にする。 |
||
77 | |||
78 | * *ステータス*: 「マージ待ち」 |
||
79 | |||
80 | 稀に、正しく意図が伝わっていなかったりバグが発生していることもある。そういう時にはチケットのフィールドを以下のように更新する。 |
||
81 | |||
82 | * *ステータス*: 「まだダメ」 |
||
83 | * *担当者*: このチケットのステータスを「レビュー待ち」に変えた人 |
||
84 | * *注記*: どういった不具合が発生しているかを詳細に説明する。UI関連ならスクリーンショットなどを貼ると良い。 |
||
85 | |||
86 | 8 | toshi_a 初音 | h3. チケットの担当者が設定されていない場合 |
87 | |||
88 | [[ロール#修正者|修正者]]以外の[[ロール#モデレータ|モデレータ]]がそのブランチをチェックアウトして動作確認を行う。 |
||
89 | |||
90 | 結果、このチケットで提案している機能が正しく実装できていることが確認できたら、ステータスを「マージ待ち」にする。 |
||
91 | |||
92 | * *ステータス*: 「マージ待ち」 |
||
93 | |||
94 | 稀に、正しく意図が伝わっていなかったりバグが発生していることもある。そういう時にはチケットのフィールドを以下のように更新する。 |
||
95 | |||
96 | * *ステータス*: 「まだダメ」 |
||
97 | * *担当者*: このチケットのステータスを「レビュー待ち」に変えた人 |
||
98 | * *注記*: どういった不具合が発生しているかを詳細に説明する。UI関連ならスクリーンショットなどを貼ると良い。 |
||
99 | |||
100 | 1 | toshi_a 初音 | h2. ステータス「マージ待ち」の場合 |
101 | |||
102 | このステータスになったチケットは、全ての修正、確認が終わっている。したがって[[ロール#コミッタ|コミッタ]]はすぐにこれをmergeしてよい。 |
||
103 | |||
104 | ただし、mergeするブランチには注意すること。次のルールに従う。 |
||
105 | |||
106 | * *developブランチから派生したブランチ* → developブランチにmergeする |
||
107 | * *タグ又はmasterから派生したブランチ* → masterブランチにmergeする |
||
108 | |||
109 | 上記のルールに従ってmasterかdevelopにmergeしたら、チケットのフィールド「ブランチ名」のブランチは削除する。 |
||
110 | |||
111 | その後、チケットのフィールドを以下のように更新する。 |
||
112 | |||
113 | * *ステータス*: 「終了」 |
||
114 | |||
115 | 7 | toshi_a 初音 | また、masterにmergeした場合は、そのmasterを更にdevelopにmergeしてもよい(see: [[リポジトリのブランチ名の規約]])。逆の、developをmasterにmergeするのは、マイナーリリースを行う時なので、toshi_a以外が行ってはならない。 |
116 | 1 | toshi_a 初音 | |
117 | h1. その他 |
||
118 | |||
119 | h2. ブランチの起点について |
||
120 | |||
121 | ブランチ名は @topic/(チケット番号)-(適当な説明)@ という書式にする(例: <code>topic/1234-fix-hogehoge</code>)。 |
||
122 | |||
123 | また、ブランチを始めるリビジョンも重要になってくる。以下のうちのどれかを選択する。可能な限り上のものを選択すること。 |
||
124 | |||
125 | * masterのHEAD |
||
126 | * developのHEAD |
||
127 | |||
128 | また、コミットの一行目の末尾には、「refs #(チケット番号)」を必ず入れること。これによって自動的にチケットとコミットが関連付けられる。 |
||
129 | |||
130 | もしPushしてからrefsの入れ忘れに気づいてしまった場合は、Redmineでそのコミットを開き、「関連するチケット」にそのチケット番号を追加すれば、手動で関連付けることができる。 |