TYPO3 Code Review Process

TYPO3 Code Reviewrgerlich. Da gibt es einen Bug

Was nun?

Projekt steht ...

TYPO3 Code ReviewBugtracker: forge.typo3.orgNichts gefunden? New IssueBug existiert: Nchstes Slide

New IssueGenaue Beschreibung, Reproduzierbarkeit, TYPO3/PHP Version, Kategorie, Screenshots

TYPO3 Code ReviewBei vielen Bugs gibt es schon fertige PatchesBug wird gemeldetPatch wird auf Review-Server geschicktVerschiedene Personen machen ein Review

Patch +2/+2: Wird gemerged und landet in Upstream. Fertig.

TYPO3 Code ReviewProblem: Fr noch mehr viele Patches gibt es keine Reviews :(

Bug ist also schon im Bugtracker eingetragenLink auf Review Server (gerrit) folgenLokalen TYPO3 source patchenKeinen lokalen Source? GIT

TYPO3 Code ReviewTYPO3 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.htmlBuch Pro Git: http://git-scm.com/book

TYPO3 Code ReviewPatch 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 ReviewTestenFunktioniert alles wie gewohntIst der Fehler behobenGibt es sonstige Probleme

So gut als mglich testen z.B. whrend der Entwicklung einer Site, evtl. sogar im Live Betrieb

TYPO3 Code ReviewCode ReviewGibt es offensichtliche Fehler im CodeIst der Code gut dokumentiert und verstndlichEntspricht er den Coding Guidelines? http://docs.typo3.org/TYPO3/CodingGuidelinesReference/

TYPO3 Code ReviewReview abgebenFr login auf review.typo3.org ist typo3.org Account notwendigButton Add comment ganz am EndeBeschreiben was und wie getestet wurdeProbleme?Verified: +1 ... Problem behoben und alles funktioniert nach wie vor wie erwartetCode Review. +1 ... Code ist lesbar und scheint kein offensichtliches Problem zu habenAuch in den diff's knnen Kommentare abgegeben werden

(c) 2013 [email protected]