Eclipse, Git und Gerrit

1. EffizienteEntwicklungsprozessemit Eclipse, Git und Gerrithttp://eclipse.org/egit+=Stefan Lay (SAP)stefan.lay@sap.comTwitter: @stefanlay

2. AgendaGit – einverteiltesVersionierungssystemGitbei Eclipse und innerhalbeinesUnternehmensCode Review mitGerritDemo: LebenszykluseinerÄnderungQ & A Effiziente Entwicklungsprozesse mit Eclipse, Git und Gerrit | © 2010 by Stefan Lay, SAP AG

3. Die HauptdarstellerGitisteinverteiltesVersionierungssystemEGitisteinEclipse Team Provider fürGithttp://www.eclipse.org/egit/JGitisteineleichtgewichtige Java-BibliothekfürGithttp://www.eclipse.org/jgit/Gerritistein Code-Review-System, basierend auf JGit http://code.google.com/p/gerrit/Effiziente Entwicklungsprozesse mit Eclipse, Git und Gerrit | © 2010 by Stefan Lay, SAP AG

4. Geschichte von Git, JGit und EGit2005 LinusTorvaldsinitiiertGit2006 Shawn Pearce initiiert JGit2009 Eclipse entscheidetsichfürGit JGit/EGit ziehen um nach eclipse.org SAP beteiligtsich 3/2010 JGit/EGit Release 0.7 (erstes Release bei Eclipse) Diff/Merge Algorithms, Automatic IP Logs 6/2010Release 0.8 (Helios) GitRepositories View, Tagging9/2010Release 0.9 (Helios SR1) Merge, Synchronize View, .gitignoreEffiziente Entwicklungsprozesse mit Eclipse, Git und Gerrit | © 2010 by Stefan Lay, SAP AG

5. 5Git vs. CVS/SVNZentralisiert

8. Langsam

9. Patches veralten

10. Merge ist problematisch

11. Verteilt

12. Historie lokal

13. Offline-Arbeit mit Versionierung

14. Schnell

15. Einfaches Rebase

17. Eclipse - RollenCommitterGewählt in einemformalenProzessKanneigeneÄnderungenohne Review committenContributorKleineÄnderungen von CommitterngereviewtGrößereÄnderungenzusätzlichformales IP review in speziellemgeschütztenBugzillaReview ToolPatches werden an BugzillaangehängtKommentare in BugzillaEffiziente Entwicklungsprozesse mit Eclipse, Git und Gerrit | © 2010 by Stefan Lay, SAP AG

19. Git @ EclipseEGit/Jgit-Entwicklung: http://egit.eclipse.orghttp://git.eclipse.org/hostetlive Eclipse Git RepositoriesVirgo, Mylyn Review, ScalaModules, SWTBot …http://dev.eclipse.org/git/index.html git mirrors für CVS Read-only Kopien, up-to-dateClonenmitgit:// oder http://“Git is the future SCM of Eclipse (Chris Aniszczyk)”Effiziente Entwicklungsprozesse mit Eclipse, Git und Gerrit | © 2010 by Stefan Lay, SAP AG

20. Git innerhalb eines Unternehmens?Git wurde für die Entwicklung des Linux Kernels konzipiertVerteilte Entwicklung bringt Vorteile für Contributors in Open-Source-ProjektenComitter / contributor model auch innerhalb eines Unternehmens -> für re-use-KomponentenProduktivität durch lokale Feature branchesGitand Gerrit ermöglichen einen (Peer) Code reviewworkflow

21. Peer Code ReviewGuido van Rossum, Google [1]When one developer writes code, another developer is asked to review that codeA careful line-by-line critique Happens in a non-threatening context Goal is cooperation, not fault-finding Often an integral part of coding processDebugging someone else's broken code– Involuntary code review: Not so good; emotions may flare[1] http://code.google.com/p/rietveld/downloads/detail?name=Mondrian2006.pdfCode Review | © 2010 by M. Sohn

22. Code Review – BenefitsGuido van Rossum, Google [1]Four eyes catch more bugsCatch bugs early to save hours of debuggingEnforce coding standardsKeep overall readability & code quality highMentoring of new developers Learn from mistakes without breaking stuffEstablish trust relationships Prepare for more delegationGood alternative to pair programmingasynchronous and across locations[1] http://code.google.com/p/rietveld/downloads/detail?name=Mondrian2006.pdfCode Review | © 2010 by M. Sohn

23. Gerrit Code ReviewGerritista ein auf JGit basierendes Code-Review-Systemhttp://code.google.com/p/gerrit/

24. DientauchalsgitServer

25. FügtZugriffskontrolleund Workflow hinzu

26. Benutzt von

27. Androidhttps://review.source.android.com/

28. JGit, EGithttp://egit.eclipse.org/r/

29. Google, SAP, …

31. Ein Branch per FeatureMaster branch enthältnurakzeptierteÄnderungenMaster verbessertsichmitjedem CommitJederFeature branch basiert auf demMaster branchStabilerStartpunktNeueste Commits in master könnenleicht in die ÄnderungintegriertwerdenGit rebase isthiersehrhilfreichEineÄnderungkannleichtverworfenwerdenKeineandereÄnderunghängt von ihrabCode Review | © 2010 by M. Sohn

34. Gerrit – LebenszykluseinerÄnderunglokaler Feature Branch

35. commit

36. push auf Gerrit

37. review

40. commit

41. push auf Gerrit

42. review

43. automatische Verifizierungtopicmaster1amastertopicVerbesserung auf Basis des Reviews

46. commit

47. push auf Gerrit

48. review

49. automatischeVerifizierungmastertopicmasterdtopic1ac3b2mastertopicVerbesserung auf Basis des Reviews

50. pushenneuer Patch Setsac13b Submit kannzuserverseitigem merge führen

52. Code Review – UnsereErfahrungenAlleÄnderungenwerdengereviewt!Ein Review kanndauern(1 Tag … Wochen) Codeautoren müssen auf den Review wartenEinparalleler Workflow istnötigJedesTeammitgliedsolltesichbeteiligenGit & GerritsindaußerordentlichhilfreichCode Review | © 2010 by M. Sohn

53. Code Review – TippsKleineÄnderungenEineÄnderungsollteatomarseinEineÄnderungsollteweder Build noch Tests brechenGrößereÄnderungensollten in eineSerie von kleinerenunterteiltwerden (patch series)- Die letzteÄnderungschaltet das Feature einDie Commit message sollte das Warumerläutern- Das Wassollteausdem Code klarhervorgehenCode Review | © 2010 by M. Sohn

55. ZusammenfassungDVCS wiez.B. GitsindleistungsstarkGitunterstützt Branches und Merging hervorragend Gitistschnell und skaliert Gerritermöglichteinen Review workflowGitund Gerriteignensichfür den EinsatzimUnternehmenUnderstanding and Using Git at Eclipse | © 2010 by C. Aniszczyk, S. Pearce, R. Rosenberg and M. Sohn

56. ResourcesAsk questions on the EGit forum or egit-dev/jgit-dev listshttp://git-scm.com/documentation is your friendIf you want comedy, watch Linus' talk at Google http://www.youtube.com/watch?v=4XpnKHJAok8Read the Pro Git book - http://progit.org/book/Understanding and Using Git at Eclipse | © 2010 by C. Aniszczyk, S. Pearce, R. Rosenberg and M. Sohn

57. Features EGit 0.8SupportedPartially supportedNot yet supported* planned for 0.9git init / git clone

58. git add

59. git status

60. git commit

61. git diff

62. git fetch

63. git log

64. git merge *

65. git rebase

66. git remote

67. git pull

68. git push

69. git stash *

70. git branch

71. git tag

72. git checkout

73. git config

74. git format-patch

75. git mv / git rm

76. git reset

77. .gitignore *

78. synchronizeview *24

79. Eclipse – Review ProcessContributorscreate patch using CVS, SVN, Git

80. attach patch to bug in BugzillaCommittersdo code and IP review

81. comment, vote in Bugzilla

82. create CQ for changes needing IP review

84. Gerrit- Workflow Every change is reviewed- Authors can invite reviewers - Complex changes reviewed by manyLook at the change- Comment on how to improve it- Discuss in context of the changeDownload the change- test it- improve itDiscussion usually leads to new improved changeCode Review | © 2010 by M. Sohn

85. Code Reviewdailyworkwithcodereviewimprovesqualityhelpslearningandavoidingsilosreviewtakes time -> parallel workflow -> githelps a lotherepeercodereviewandautomatedverification on isolatedchange -> voting -> improvechangebased on comments -> submittomaster -> centralbuildandtest

Eclipse, Git und Gerrit

Weitere ähnliche Inhalte

Was ist angesagt? (6)

Andere mochten auch (15)

Ähnlich wie Eclipse, Git und Gerrit (13)

Eclipse, Git und Gerrit