Directly on code review tool.

22%

Do pull, review and test locally

22%

Both

56%

What's the best (or correct) way to perform a code review and why?

  1. Open SVC (Github, Bitbucket, Gitlab.. or any code review tool) and look into code and make some comments

  2. Pull the source into your machine, check the code, made some basic test, and make some comments on code review tool

Write your answer…

7 answers

You need an option for both! I look at the code visually in github before pulling it down because sometimes I can reject it before spending the time and energy pulling it down to test locally.

Show all replies

Not sure if I can post links in here but this is the first draft I wrote up for doing Code Reviews:

codingwithcoffee.net/guides/code-reviews

Reply to this…

Hashnode is a friendly and inclusive dev community.
Come jump on the bandwagon!

  • 💬 Ask programming questions without being judged

  • 🧠 Stay in the loop and grow your knowledge

  • 🍕 More than 500K developers share programming wisdom here

  • ❤️ Support the growing dev community!

Create my profile

By the way, most replies (including mine) were centered around the 'where' (online, or pull). But since the title is 'how', I think it's a good opportunity to share this good talk from Nina Zakharenko. Most of it is really generic and language agnostic, and 90% of the talk will still be relevant to people not doing any Python at all.

youtube.com/watch?v=6L3ZVLtSeo8

How to be a good PR reviewer. And how to be a good PR submitter. (Spoiler; both could be hard, because of people, but nothing a walk or a coffee break couldn't resolve)

Reply to this…

Like Jake wrote, I'd vote for both. Depending on the codebase, the RP size and the author, I could only review it online, or pull it to review locally, or both...

Reply to this…

I prefer to pull as I dislike the idea of working on live codebases -- BUT local testing often doesn't line up with what a live copy spits out when running due to server differences, so I would actually say both.

Though as a rule of thumb I don't like version control software and avoid it, I prefer dealing with environments where you have a project manager that does their flipping job! the past decade and a half as VCS has been thrown at EVERYTHING I've watched far too many codebases get tanked by it thanks to the lack of direct communication between the people working on things, lack of oversight, and lack of any real chain of command.

I'm not saying it shouldn't be used or doesn't have a place, but unless you have a benevolent dictator sitting atop things riding herd, far FAR too often it can be as bad if not worse than going without.

Reply to this…

At work, I use a code review tool, without pulling in the code except in special cases. We have testers who will verify that it works; the review is mostly for conventions, code quality, architecture, seeing if things make sense... If it needs running the code, it's the job of the developer and the tester.

For open source projects, I don't have testers, so I more often pull the code to test locally. This is besides using the code review interface of Github / Bitbucket.

Perhaps if I'm ever in charge of this choice, I'd look for a code review tool with good IDE integration. I'd like to test code and leave comments without leaving the IDE.

Show all replies

It doesn't mean that for me. If I approve a code review, it means it's okay to be tested by the QA team. Only then is it merged.

Reply to this…

Load more responses

The Author Card

Riderman de Sousa Barbosa's photo

Riderman de Sousa Barbosa

Frontend Developer

Appreciations

24

Location

Belo Horizonte, MG - Brasil

Joined

Feb 2, 2016