TYPO3 Code Review Process

  • Published on
    19-Jul-2015

  • View
    402

  • Download
    1

Embed Size (px)

Transcript

<ul><li><p>TYPO3 Code Reviewrgerlich. Da gibt es einen Bug</p><p>Was nun?</p><p>Projekt steht ...</p></li><li><p>TYPO3 Code ReviewBugtracker: forge.typo3.orgNichts gefunden? New IssueBug existiert: Nchstes Slide</p><p>New IssueGenaue Beschreibung, Reproduzierbarkeit, TYPO3/PHP Version, Kategorie, Screenshots</p></li><li><p>TYPO3 Code ReviewBei vielen Bugs gibt es schon fertige PatchesBug wird gemeldetPatch wird auf Review-Server geschicktVerschiedene Personen machen ein Review</p><p>Patch +2/+2: Wird gemerged und landet in Upstream. Fertig.</p></li><li><p>TYPO3 Code ReviewProblem: Fr noch mehr viele Patches gibt es keine Reviews :(</p><p>Bug ist also schon im Bugtracker eingetragenLink auf Review Server (gerrit) folgenLokalen TYPO3 source patchenKeinen lokalen Source? GIT</p></li><li><p>TYPO3 Code ReviewTYPO3 master aus GIT beziehen: cd /home/kraftb/src/ git clone git://git.typo3.org/Packages/TYPO3.CMS.git</p><p>GIT Cheatsheet: http://www.ndpsoftware.com/git-cheatsheet.htmlBuch Pro Git: http://git-scm.com/book </p></li><li><p>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 &amp;&amp; git cherry-pick FETCH_HEAD</p></li><li><p>TYPO3 Code ReviewTestenFunktioniert alles wie gewohntIst der Fehler behobenGibt es sonstige Probleme</p><p>So gut als mglich testen z.B. whrend der Entwicklung einer Site, evtl. sogar im Live Betrieb</p></li><li><p>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/</p></li><li><p>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</p></li><li><p>(c) 2013 kraft@webconsulting.at</p></li></ul>

Recommended

View more >