SlideShare ist ein Scribd-Unternehmen logo
1 von 19
Downloaden Sie, um offline zu lesen
Put more eyes on code




Dariusz Łuksza
Blog: http://luksza.org
Mail: dariusz@luksza.org
?
Blog: http://luksza.org
Mail: dariusz@luksza.org
Cel!




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


Blog: http://luksza.org
Mail: dariusz@luksza.org
Jak powinno wyglądać code review                              ?




                           Autor http://www.stellman-greene.com
Blog: http://luksza.org
Mail: dariusz@luksza.org
A może tak   ?




Blog: http://luksza.org
Mail: dariusz@luksza.org
Raczej tak:                                              Tu powinno być
                                                             get_column()




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




Blog: http://luksza.org
Mail: dariusz@luksza.org
Ale z czego korzystać?




                       1.0
Blog: http://luksza.org
Mail: dariusz@luksza.org
Ale z czego korzystać?




                           EGit   Gerrit




Blog: http://luksza.org
Mail: dariusz@luksza.org
It's a nightly build !!!




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




Blog: http://luksza.org
Mail: dariusz@luksza.org
Słowniczek
    ●   commit
    ●   branch
    ●   merge
    ●   rebase
    ●   fetch/pull
    ●   push
    ●   patch/change set


Blog: http://luksza.org
Mail: dariusz@luksza.org
Ale z czego korzystać?




                     Demo
Blog: http://luksza.org
Mail: dariusz@luksza.org
Ale z czego korzystać?




                       2.0
Blog: http://luksza.org
Mail: dariusz@luksza.org
Ale z czego korzystać?




Blog: http://luksza.org
Mail: dariusz@luksza.org
Ale z czego korzystać?




                     Demo
Blog: http://luksza.org
Mail: dariusz@luksza.org
Co zyskujemy?
    ●   Wyrównuje poziom wiedzy
    ●   Redukuje duplikację kodu
    ●   Nie psuje buildów
    ●   Podnosi jakość kodu




Blog: http://luksza.org
Mail: 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.org
Mail: dariusz@luksza.org
Wszystko zależy od ludzi




                           Autor http://spam.blog.com/




Blog: http://luksza.org
Mail: dariusz@luksza.org
Q&A?
Blog: http://luksza.org
Mail: 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.org
Mail: dariusz@luksza.org

Weitere ähnliche Inhalte

Mehr von Dariusz Łuksza

A story of 715 commits… in ~15 minutes

A story of 715 commits… in ~15 minutes
A story of 715 commits… in ~15 minutes

A story of 715 commits… in ~15 minutes
Dariusz Łuksza
 
Your own full blown Gerrit plugin
Your own full blown Gerrit pluginYour own full blown Gerrit plugin
Your own full blown Gerrit pluginDariusz Łuksza
 
Review your code like a Googler
Review your code like a GooglerReview your code like a Googler
Review your code like a GooglerDariusz Łuksza
 
Gerrit JavaScript Plugins
Gerrit JavaScript PluginsGerrit JavaScript Plugins
Gerrit JavaScript PluginsDariusz Łuksza
 
Eclipse of idleness and focus on current task (rev. 2)
Eclipse of idleness and focus on current task (rev. 2)Eclipse of idleness and focus on current task (rev. 2)
Eclipse of idleness and focus on current task (rev. 2)Dariusz Łuksza
 

Mehr von Dariusz Łuksza (7)

A story of 715 commits… in ~15 minutes

A story of 715 commits… in ~15 minutes
A story of 715 commits… in ~15 minutes

A story of 715 commits… in ~15 minutes

 
One Man App
One Man AppOne Man App
One Man App
 
Your own full blown Gerrit plugin
Your own full blown Gerrit pluginYour own full blown Gerrit plugin
Your own full blown Gerrit plugin
 
Review your code like a Googler
Review your code like a GooglerReview your code like a Googler
Review your code like a Googler
 
Git in Eclipse
Git in EclipseGit in Eclipse
Git in Eclipse
 
Gerrit JavaScript Plugins
Gerrit JavaScript PluginsGerrit JavaScript Plugins
Gerrit JavaScript Plugins
 
Eclipse of idleness and focus on current task (rev. 2)
Eclipse of idleness and focus on current task (rev. 2)Eclipse of idleness and focus on current task (rev. 2)
Eclipse of idleness and focus on current task (rev. 2)
 

Put more eyes on code

  • 1. Put more eyes on code Dariusz Łuksza Blog: http://luksza.org Mail: dariusz@luksza.org
  • 3. Cel! Autor http://www.flickr.com/photos/bogdansuditu/ Blog: http://luksza.org Mail: dariusz@luksza.org
  • 4. Jak powinno wyglądać code review ? Autor http://www.stellman-greene.com Blog: http://luksza.org Mail: dariusz@luksza.org
  • 5. A może tak ? Blog: http://luksza.org Mail: dariusz@luksza.org
  • 6. Raczej tak: Tu powinno być get_column() Autor http://www.flickr.com/photos/sebastian_bergmann/ Blog: http://luksza.org Mail: dariusz@luksza.org
  • 7. Ale z czego korzystać? 1.0 Blog: http://luksza.org Mail: dariusz@luksza.org
  • 8. Ale z czego korzystać? EGit Gerrit Blog: http://luksza.org Mail: dariusz@luksza.org
  • 9. It's a nightly build !!! Znaleziono na http://vyanessa.blogspot.com/ Blog: http://luksza.org Mail: dariusz@luksza.org
  • 10. Słowniczek ● commit ● branch ● merge ● rebase ● fetch/pull ● push ● patch/change set Blog: http://luksza.org Mail: dariusz@luksza.org
  • 11. Ale z czego korzystać? Demo Blog: http://luksza.org Mail: dariusz@luksza.org
  • 12. Ale z czego korzystać? 2.0 Blog: http://luksza.org Mail: dariusz@luksza.org
  • 13. Ale z czego korzystać? Blog: http://luksza.org Mail: dariusz@luksza.org
  • 14. Ale z czego korzystać? Demo Blog: http://luksza.org Mail: dariusz@luksza.org
  • 15. Co zyskujemy? ● Wyrównuje poziom wiedzy ● Redukuje duplikację kodu ● Nie psuje buildów ● Podnosi jakość kodu Blog: http://luksza.org Mail: dariusz@luksza.org
  • 16. 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.org Mail: dariusz@luksza.org
  • 17. Wszystko zależy od ludzi Autor http://spam.blog.com/ Blog: http://luksza.org Mail: dariusz@luksza.org
  • 19. 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.org Mail: dariusz@luksza.org