The reviewer checklist.

Nicola Pietroluongo

Friday 22 September 2017 from 15:25 to 15:37

Talk in English - UK at SymfonyLive London 2017
Track Name: Track B
View Slides: https://www.slideshare.net/secret/tec6DHgdsKIfCs
Short URL: https://joind.in/talk/20dcd (QR-Code (opens in new window))

Avg. Rating

The reviewer checklist - or better said how to be a better code reviewer with a methodical approach. In this talk we explore what a code reviewer should flag out and what should not, from finding bad code smell, design problem to spot performance or security issue and poor test coverage. The talks includes snippets of code to demonstrate solutions, challenges I’ve encountered while reviewing code in several big projects and, of course, what has become my checklist.

Comments

Comments are closed.

Michael Gwynne at 10:04 on 26 Sep 2017

Useful talk. Centred more on CI review tools more than I expected but still really useful.

Anonymous at 10:00 on 30 Sep 2017

Interesting talk about what code review really is. Few useful tips and good presenter.

Richard Lewis at 10:32 on 2 Oct 2017

Great tips on how to speed up and improve code review.

Thomas Meehan at 11:52 on 2 Oct 2017

Really good talk. Helped to get my head around the importance of code review. Provided some good tips as well.

Daniel Noel-Davies at 21:54 on 4 Oct 2017

Very good talk, gave my colleagues some great ideas to take back to the workplace and try to implement and make the process nicer to work with.

Thanks!