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