typo3 code review process
TRANSCRIPT
TYPO3 Code Review
Ärgerlich. Da gibt es einen Bug
Was nun?
Projekt steht ...
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
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.
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
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
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
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
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/
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 haben
Auch in den „diff's“ können Kommentare abgegeben werden
(c) 2013 [email protected]