Click here to load reader
View
407
Download
1
Embed Size (px)
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]