TYPO3 Code Review Process

  • Published on
    19-Jul-2015

  • View
    402

  • Download
    1

Embed Size (px)

Transcript

  • 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 kraft@webconsulting.at