Skip to content

Latest commit

 

History

History
82 lines (57 loc) · 2.05 KB

README.md

File metadata and controls

82 lines (57 loc) · 2.05 KB
Raw

kanazawaApp-2019

stockSupporter

注意事項

  • master で作業しない。
  • conflict 起こさないように master 更新されたらローカルの方も更新する。
  • 自分が変更したファイル以外は極力上げない。
  • ブランチ名はキャメルケースを使う。
  • コミット名は日本語で何をしたのかわかりやすくつける。
  • Pull Request(PR)と Issue を関連づける。

命名規則

ブランチ名

  • 画面を作る場合
createScreen-hoge-yourName
  • やることリストをやった場合
toDo-hoge
  • 何かを修正した場合
fix-hoge
  • 何か機能を追加した場合
add-hoge

ID 名

  • component の ID
android:id="@+id/lowerCamelCase
  • app/src/main/res/values 下の xml ファイルに記述する ID(string.xml の例)
<string name="lowerCamelCase">文字列</string>

ファイル名

  • app/src/main/res/drawable 下のファイル名(png ファイルの例)
snake_case.png

開発手順

  • Issue を立てる
  • 担当している Issue に Assignees をつける
  • Issue を関連付けた Pull Request をたてる
  • Pull Request に Assignees と Label をつける
    • 作業中なら「作業中」
    • 作業が完了したなら「レビュー依頼」
  • レビューしてもらう
  • マージ

「ラベル」の使い方

  1. レビューを貰う人が PR を出して「レビュー依頼」を貼る。
  2. 誰か 1 人のレビュワーがレビューした段階で「レビュー依頼」を外す。同時に、”見ました”の意味で「reviewed」を貼る。
  3. レビューを貰った人が、指摘部分を修正してコミット。「reviewed」を外し、再び「レビュー依頼」を貼る。
  4. 2 と 3 を n 回(n∈N)繰り返し、レビュワーが問題無いと判断した場合 Approve する。
  5. 任意の人数(例えば 2 人)が Approve したら「Approved」を貼る。(最後に Approve した人が貼る。)
  6. 「Approved」が貼られた PR を merge する。