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 を確認してみるに以下なソレを確認。

github push branch

で、Compare & pull request なボタンを click すると以下が出てきます。

github compare and pull request

あるいはcompare & review なボタン (左にある緑のヤツ) を click するとこんなナニが

github compare and merge

push した branch をレビュするには branch を push したソレに切り替えて

github remote branch

緑のボタンか。

github review branch

これで diff を確認できるのか。pull request してみるとどうなるのかな。

github 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 を複数にして云々してみる方向で。