Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.
Gerrit Code Review 教學
林軒宇 2014.04.01
Coding convention
每個縮排 4 space (不要⽤用tab)
<img> 不要給 width, height
CSS 要⽤用 selector 縮⼩小範圍
路徑⽤用相對路徑,不要⽤用絕對路徑
Code Review
網址 http://review.xxx.com
commit message 要⽤用英⽂文 (因為 Gerrit 不⽀支援中⽂文)。︒
每個 patch 要保持簡短利落,commit message 要包含 JIRA
...
Gerrit 設定
先在 Windows 裝 git
http://blog.crboy.net/2012/05/git-on-windows.html
要先 signup
設定 SSH Public Keys
設定 Username
SSH Public key 設定(僅供參考)
開啓 Git Bash
ssh-keygen
預設都按 enter
把 C:Users你的⼤大名.sshid_rsa 的內容複製到 Gerrit 網⾴頁
上的 SSH Public key
或可參...
Gerrit Q & A
遇到 Permission denied (publickey). 這類的問題,請確認
~/.gitconfig 和 ~/.ssh/config 是否都已設定 User name 和
email:
第⼀一次抓 code
repo init -u ssh://在網⾴頁上設定的Username@xxx.com:29418/
ih/manifests.git
repo sync
開始新的⼀一個 branch
repo start 其實是開⼀一個新的 branch
查看⽬目前狀況
確定要 commit 的檔案可⽤用 git add
綠⾊色代表 git commit 會被 commit 的檔案,紅⾊色代表還沒準備
要 commit (⽤用 git commit -a 則可不管紅綠都會⼀一起 commit)
上傳前記得先 git diff
有紅⾊色的地⽅方代表是 tab,要把它改成 space
上傳到 gerrit
code review 時可順便找 tab
Code-review 分數的意義
+2 這個 patch 沒問題,驗證過,可以直接 submit。︒
+1 這個 patch 沒問題,驗證過,但需要其他⼈人幫忙驗證。︒
+0 留⾔言專⽤用。︒
-1 這個 patch 有問題,(不論是否驗證過...
⽤用 --amend 來修改 patch
在還沒有被 merge 前,同⼀一個 patch 都可⽤用 --amend 來修改
--amend 會出現上次的 log
此時可以修改 commit message,不修改則代表沿⽤用上次寫的
message
再度上傳到 Gerrit
刪除檔案的⽅方式
merge 之後⋯⋯
可以⽤用 repo sync 來抓 server 上所有最新的 code
DOREMI
git, repo, Gerrit 基礎教學
git, repo, Gerrit 基礎教學
git, repo, Gerrit 基礎教學
Upcoming SlideShare
Loading in …5
×

git, repo, Gerrit 基礎教學

16,534 views

Published on

git, repo, Gerrit 基礎教學

  1. 1. Gerrit Code Review 教學 林軒宇 2014.04.01
  2. 2. Coding convention 每個縮排 4 space (不要⽤用tab) <img> 不要給 width, height CSS 要⽤用 selector 縮⼩小範圍 路徑⽤用相對路徑,不要⽤用絕對路徑
  3. 3. Code Review 網址 http://review.xxx.com commit message 要⽤用英⽂文 (因為 Gerrit 不⽀支援中⽂文)。︒ 每個 patch 要保持簡短利落,commit message 要包含 JIRA id(如果有的話),commit message 若需要⽤用比較⼤大的篇幅敘 述,則在 JIRA 補充。︒ 只要完成⼀一⼩小部分功能,即可當作⼀一個 patch 上傳,可以讓未 來的 debug 變簡單,⽽而不要實作⼀一堆功能後再⼀一次上傳。︒
  4. 4. Gerrit 設定 先在 Windows 裝 git http://blog.crboy.net/2012/05/git-on-windows.html 要先 signup 設定 SSH Public Keys 設定 Username
  5. 5. SSH Public key 設定(僅供參考) 開啓 Git Bash ssh-keygen 預設都按 enter 把 C:Users你的⼤大名.sshid_rsa 的內容複製到 Gerrit 網⾴頁 上的 SSH Public key 或可參考 https://help.github.com/articles/generating-ssh- keys#platform-windows
  6. 6. Gerrit Q & A 遇到 Permission denied (publickey). 這類的問題,請確認 ~/.gitconfig 和 ~/.ssh/config 是否都已設定 User name 和 email:
  7. 7. 第⼀一次抓 code repo init -u ssh://在網⾴頁上設定的Username@xxx.com:29418/ ih/manifests.git repo sync
  8. 8. 開始新的⼀一個 branch repo start 其實是開⼀一個新的 branch
  9. 9. 查看⽬目前狀況
  10. 10. 確定要 commit 的檔案可⽤用 git add 綠⾊色代表 git commit 會被 commit 的檔案,紅⾊色代表還沒準備 要 commit (⽤用 git commit -a 則可不管紅綠都會⼀一起 commit)
  11. 11. 上傳前記得先 git diff 有紅⾊色的地⽅方代表是 tab,要把它改成 space
  12. 12. 上傳到 gerrit
  13. 13. code review 時可順便找 tab
  14. 14. Code-review 分數的意義 +2 這個 patch 沒問題,驗證過,可以直接 submit。︒ +1 這個 patch 沒問題,驗證過,但需要其他⼈人幫忙驗證。︒ +0 留⾔言專⽤用。︒ -1 這個 patch 有問題,(不論是否驗證過,可能只是單純 coding style 的問題),請在留⾔言詳述。︒ -2 這個 patch 有問題,⽽而且可能連驗證都無法通過,(例如功能 有錯誤,或是沒有 rebase 等等),此時除非留下 -2 的 reviewer 再重新評分,否則此 patch 無法被 submit。︒
  15. 15. ⽤用 --amend 來修改 patch 在還沒有被 merge 前,同⼀一個 patch 都可⽤用 --amend 來修改
  16. 16. --amend 會出現上次的 log 此時可以修改 commit message,不修改則代表沿⽤用上次寫的 message
  17. 17. 再度上傳到 Gerrit
  18. 18. 刪除檔案的⽅方式
  19. 19. merge 之後⋯⋯ 可以⽤用 repo sync 來抓 server 上所有最新的 code DOREMI

×