During PR review - do you check in into the branch?

github logo ・1 min read

Have found a very fundamental post on Code review https://charcoalbin.com/posts/code-review.html where Authour states that it is needed to check the functionality of PR.

Usability Test
Confirm if the PR achieves its goal, visually and functionally.

I have thought about this idea, but actually never tried, mostly because of time.
Do some of you follow it, what's your perspective?

twitter logo DISCUSS (3)
markdown guide
 

To be honest no - I've never done that, nor have I seen someone do that. It would definitely increase PR level (and possible bugs and/or missing specs), but usually I trust the developer to validate what he wrote with whomever is in charge of the product, allowing the PR to focus on code quality, rather than product quality.

 

Most of the time.

I think checking out the branch in local and testing if the feature works should be applied for every PR that either fixes a bug, introduces a new feature or is a refactor. It's not that I don't trust my coworkers but I think all humans are fallible and are allowed to miss some things.

What I don't trust is my ability to statically check all possible changes that the code will introduce just by looking at the PR diff.

 

Sometimes, if the PR is big or contains big UI changes or if I just feel like it. I'm not following any strict rules on that.

I use hub pr checkout command and then test it.

Classic DEV Post from Jan 31

What are the least intuitive fundamentals and best practices in software development?

Some things we do kind of make sense in and of themself. Some things have evolv...

Oles Maiboroda profile image
web, javascript