typo3 code review process

13
TYPO3 Code Review Ärgerlich. Da gibt es einen Bug Was nun? Projekt steht ...

Upload: bernhard-kraft

Post on 19-Jul-2015

414 views

Category:

Technology


1 download

TRANSCRIPT

Page 1: TYPO3 Code Review Process

TYPO3 Code Review

Ärgerlich. Da gibt es einen Bug

Was nun?

Projekt steht ...

Page 2: TYPO3 Code Review Process
Page 3: TYPO3 Code Review Process

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

Page 4: TYPO3 Code Review Process

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.

Page 5: TYPO3 Code Review Process

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

Page 6: TYPO3 Code Review Process

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

Page 7: TYPO3 Code Review Process
Page 8: TYPO3 Code Review Process

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

Page 9: TYPO3 Code Review Process
Page 10: TYPO3 Code Review Process

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

Page 11: TYPO3 Code Review Process

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/

Page 12: TYPO3 Code Review Process

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

Page 13: TYPO3 Code Review Process

(c) 2013 [email protected]