Ciao!
Mi farebbe piacere sapere come applicate code reviews da voi. Ogni azienda/team le fa in modo diverso, per alcuni si tratta di "spiegare" il diff tra il nuovo codice e il vecchio - con live sessions per ogni feature. Alcuni team possono essere sono piu' puntigliosi e vanno a cercare le virgole fuori posto.
Voi cosa cercate in una code review? Cosa fate? E soprattutto, come date feedback ai vostri colleghi (e non)?
Io generalmente guardo al codice, cerco di capire cosa fa (magari testo sulla mia macchina), pongo domande piu' che affermazioni, e a meno che non ci siano vistosi errori di programmazione, solitamente non chiedo granche'. Usiamo tool automatici per prevenire errori di formatting e altro, cosi' che le code reviews si soffermino di piu' sul codice che sulle virgole fuori posto. Inoltre, verifico che i test abbiano senso e non siano ripetitivi, ecc. Ah, spero che non facciate review di design nelle code reviews ...
E voi?