Se ha denunciado esta presentación.
Utilizamos tu perfil de LinkedIn y tus datos de actividad para personalizar los anuncios y mostrarte publicidad más relevante. Puedes cambiar tus preferencias de publicidad en cualquier momento.
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 基礎教學
Próxima SlideShare
Cargando en…5
×

git, repo, Gerrit 基礎教學

16.590 visualizaciones

Publicado el

  • Sé el primero en comentar

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

×