Pedometer に手を入れてみるなど
帰宅しておやつを食べつつ燃料注入を開始して clone するなど。
$ git clone git@github.com:yamanetoshi/Pedometer.git
で、branch を作って
$ git checkout -b 2version4
.gitignore に手を入れたり SDK なバージョンを 4.0.3 にしたりなど。
つうか
自分が作ったものらしいのですが中身掘削から開始しなければならない模様です。これから確認。今日はこの後、掘削中心になるのかどうなのか。ざっくり見渡してみたところ
- 何故か MENU で LAUNCHER な Activity が二つある
- Service と BroadcastReceiver が居る
- Provider も定義されてる
- これ、今使ってる方式で書換え必要ですね
なカンジ。ちょい色々整理させて頂きつつ
- Provider な実装は書換え
- Service 落ちたら再起動する
あたりを盛り込むかどうか。
とりあえず
餃子製造に着手せねば、らしいので一旦中断。
再開
Manifest と .gitignore を修正した状態で commit を作成して branch を push してみることに。
$ git push origin 2version4
と brwoser で repository を確認してみるに以下なソレを確認。
で、Compare & pull request なボタンを click すると以下が出てきます。
あるいはcompare & review なボタン (左にある緑のヤツ) を click するとこんなナニが
push した branch をレビュするには branch を push したソレに切り替えて
緑のボタンか。
これで diff を確認できるのか。pull request してみるとどうなるのかな。
diff 確認できます。とりあえずこの pull request は merge しときます。で、ローカル側に修正を取得してみます。
$ git fetch remote: Counting objects: 1, done. remote: Total 1 (delta 0), reused 0 (delta 0) Unpacking objects: 100% (1/1), done. From github.com:yamanetoshi/Pedometer 7198d61..881c03f master -> origin/master $ git merge origin/master Updating 7198d61..881c03f Fast-forward .gitignore | 3 +++ AndroidManifest.xml | 1 + 2 files changed, 4 insertions(+) $
これ、修正量は少ない方が見る側てきに嬉しいのですが、以降に控えてる修正の量はそれなりに多いですね。ちなみに merge 済みの branch はすぐには確認できない模様。merge 済みのカスは delete しといた方が良いのかどうか。
追記
ローカル側で git log 見てみた。
$ git log commit 881c03f03631b1ffda602976903389ebfdf191d0 Merge: 7198d61 6b423bf Author: YAMANE Toshiaki <yamanetoshi@gmail.com> Date: Mon Jul 15 02:25:06 2013 -0700 Merge pull request #1 from yamanetoshi/2version4 Modify Android SDK version to 4.0.3 (Level15) commit 6b423bf94452e105749a310eba2bdabcb9e30b9e Author: YAMANE Toshiaki <yamanetoshi@gmail.com> Date: Mon Jul 15 17:40:33 2013 +0900 Modify Android SDK version to 4.0.3 (Level15) commit 7198d61f00ef0692ae09b2ba2a29f8cee2a1ae43 Author: yamanetoshi <yamanetoshi@gmail.com> Date: Wed Nov 23 19:46:36 2011 +0900 initial commit $
これ、rebase した覚えはないんですが、pull request したら纏まるの? って思ったら最初から commit は一つだったorz
次回は意図的に commit を複数にして云々してみる方向で。