RSS Feed LinkedIn Google Plus

Címke: Andrew Berry

  1. A kód felülvizsgálatról

    2012. február 3. péntek

    A code review ugyanolyan fontos része a mindennapi fejlesztési folyamatnak, mint a QA, automatizált tesztelés és a dokumentáció. Andrew Berry, a Drupal csapat tagja részletesen összeírta, miket szokott vizsgálni:

    • A kód története (miért volt szükséges változtatni, a commit – vagy merge – atomisága, stb.)
    • A rendszer APIk megfelelő használata (pl: azon az absztrakciós szinten oldjuk meg a problémát, ahol kell)
    • Biztonság
    • API központú tervezés
    • Dokumentáció
    • Unit és funkcionális tesztek
    • Kódolási stílus betartása

    Code reviews are an acknowledgement that every developer is a human being, and humans make mistakes. No matter the skill or background of a developer, reviewing their code can only improve the final product. […]

    While contributed modules don’t often have the resources for full code reviews of every patch, be sure that any module author would love reviews of their code.