TYPO3 Code Review Process

653 Aufrufe

Veröffentlicht am

Präsentation für das TUGA 35 Meeting. Es wird erklärt wie der Code Review von Patches und Features für TYPO3 abläuft und wie jeder mithelfen kann.

Veröffentlicht in: Technologie
0 Kommentare
0 Gefällt mir
Statistik
Notizen
  • Als Erste(r) kommentieren

  • Gehören Sie zu den Ersten, denen das gefällt!

Keine Downloads
Aufrufe
Aufrufe insgesamt
653
Auf SlideShare
0
Aus Einbettungen
0
Anzahl an Einbettungen
1
Aktionen
Geteilt
0
Downloads
1
Kommentare
0
Gefällt mir
0
Einbettungen 0
Keine Einbettungen

Keine Notizen für die Folie

TYPO3 Code Review Process

  1. 1. TYPO3 Code Review Ärgerlich. Da gibt es einen Bug Was nun? Projekt steht ...
  2. 2. TYPO3 Code Review  Bugtracker: forge.typo3.org  Nichts gefunden? „New Issue“  Bug existiert: Nächstes Slide „New Issue“ Genaue Beschreibung, Reproduzierbarkeit, TYPO3/PHP Version, Kategorie, Screenshots
  3. 3. TYPO3 Code Review Bei vielen Bugs gibt es schon fertige Patches  Bug wird gemeldet  Patch wird auf Review-Server geschickt  Verschiedene Personen machen ein Review  Patch +2/+2: Wird gemerged und landet in „Upstream“. Fertig.
  4. 4. TYPO3 Code Review Problem: Für noch mehr viele Patches gibt es keine Reviews :(  Bug ist also schon im Bugtracker eingetragen  Link auf Review Server (gerrit) folgen  Lokalen TYPO3 source patchen  Keinen lokalen Source? GIT
  5. 5. TYPO3 Code Review  TYPO3 master aus GIT beziehen: cd /home/kraftb/src/ git clone git://git.typo3.org/Packages/TYPO3.CMS.git  GIT Cheatsheet: http://www.ndpsoftware.com/git-cheatsheet.html  Buch „Pro Git“: http://git-scm.com/book
  6. 6. TYPO3 Code Review  Patch von gerrit auf lokalen Souce anwenden: cd TYPO3.CMS git fetch git://git.typo3.org/Packages/TYPO3.CMS refs/changes/39/25839/2 && git cherry-pick FETCH_HEAD
  7. 7. TYPO3 Code Review Testen  Funktioniert alles wie gewohnt  Ist der Fehler behoben  Gibt es sonstige Probleme  So gut als möglich testen z.B. während der Entwicklung einer Site, evtl. sogar im Live Betrieb
  8. 8. TYPO3 Code Review Code Review    Gibt es offensichtliche Fehler im Code Ist der Code gut dokumentiert und verständlich Entspricht er den Coding Guidelines? http://docs.typo3.org/TYPO3/CodingGuidelinesReference/
  9. 9. TYPO3 Code Review Review abgeben  Für login auf review.typo3.org ist typo3.org Account notwendig  Button „Add comment“ ganz am Ende  Beschreiben was und wie getestet wurde  Probleme?   Verified: +1 ... Problem behoben und alles funktioniert nach wie vor wie erwartet Code Review. +1 ... Code ist lesbar und scheint kein offensichtliches Problem zu
  10. 10. (c) 2013 kraft@webconsulting.at

×