Put more eyes on code

Dariusz ŁukszaBlog: http://luksza.orgMail: dariusz@luksza.org

?Blog: http://luksza.orgMail: dariusz@luksza.org

Cel!

Blog: http://luksza.orgMail: dariusz@luksza.org

Autor http://www.flickr.com/photos/bogdansuditu/

Jak powinno wyglądać code review?

Blog: http://luksza.orgMail: dariusz@luksza.org

Autor http://www.stellman-greene.com

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

A może tak?

Blog: http://luksza.orgMail: dariusz@luksza.org

Raczej tak:

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

Tu powinno byćget_column()

Autor http://www.flickr.com/photos/sebastian_bergmann/

Ale z czego korzystać?

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

1.0

Ale z czego korzystać?

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

EGit

Blog: http://luksza.orgMail: dariusz@luksza.org

Gerrit

It's a nightly build !!!

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

Znaleziono na http://vyanessa.blogspot.com/

Słowniczek● commit● branch● merge● rebase● fetch/pull● push● patch/change set

Blog: http://luksza.orgMail: dariusz@luksza.org

Ale z czego korzystać?

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

Demo

Ale z czego korzystać?

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

2.0

Ale z czego korzystać?

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

Ale z czego korzystać?

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

Demo

Co zyskujemy?● Wyrównuje poziom wiedzy● Redukuje duplikację kodu● Nie psuje buildów● Podnosi jakość kodu

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

Co zyskujemy?● Wyrównuje poziom wiedzy● Redukuje duplikację kodu● Nie psuje buildów● Podnosi jakość kodu

● Spowalnia development● Wszystko zależy od ludzi!!

Blog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.orgBlog: http://luksza.orgMail: dariusz@luksza.org

Wszystko zależy od ludzi

Autor http://spam.blog.com/

Blog: http://luksza.orgMail: dariusz@luksza.org

Q&A?Blog: http://luksza.orgMail: dariusz@luksza.org

A dokładniej ?● EGit * – http://eclipse.org/egit● Myly-reviews * – http://eclipse.org/reviews● Gerrit – http://code.google.com/p/gerrit/● Gerrit trigger – https://wiki.jenkins-ci.org/display/JENKINS/Gerrit+Trigger

● * nightly builds z http://hudson.eclipse.org/

Blog: http://luksza.orgMail: dariusz@luksza.org