Code Review

We've created a new screencast outlining some of the best practices that apply to performing code reviews, and how Upsource can help apply those best practices. In this blog post we've also transcribed the content, and have provided links to further information. https://www.youtube.com/embed/EjwD…

We've previously covered at What to Look for in Java 8 Code, now Java is moving faster than ever it's time to do an update and cover what to look for in Java 9 code. While Java 9 has even now been replaced with Java 10, and Java 11 in coming in September, these Java 9 features are, of course, availa…

Code review like no other development practice relies on humans being efficient. And for most of us to achieve that, it's important to have our tasks well organized and our time well planned. Upsource takes care of a lot of things for you without any need to configure anything. For example, if y…

The last three posts on code review workflows show that there's more than one way to do a code review, and that the approach you take will depend upon what you want to achieve with your reviews and the type of team you're working with. In this post, we'll cover how Upsource aims to make your work…

In the last two posts we talked a bit about how effective those approaches are for the author to learn and improve, because those two approaches focus on getting the code correct. The Design Evolution approach additionally mentions the value of the whole team seeing the evolution of the code. Thi…

In the last post we covered a review process where the code review is performed after all the code is written with the goal of approving the suggested changes. If getting the design right is important to your team, then a collaborative and iterative approach may be appropriate — the code can be c…

We've talked a lot about what to look for in a Code Review, but we haven't covered the process at all. There are many different reasons to perform code reviews in your team or organisation, and the process you choose should be driven by what you're trying to achieve with the reviews. The first ex…

In earlier blog posts we've talked about general things to look for during a code review. Recently I've been working a lot with Java 8, and I'm starting to have some opinions on what modern Java code should look like.  I've written about some of my Top Tips from an IntelliJ IDEA point of view, but I…

Code review workflows are as diverse as teams that use them. Some teams are flexible and review only certain commits as they see fit, others have more formal criteria of what should be reviewed, by whom, and how the outcome of the code review should affect the development process. For example, there…

Last year we published a series of blog posts written by talented Trisha Gee attempting to answer questions like: What should we look for in a code review? What kind of questions we should ask ourselves as reviewers? What kind of problems are worth our time? Given how vast is the topic, we've ba…

We’ve had two themes running through the articles on what to look for in a code review: Lots of what we’re tempted to look for can (and should) be automated Humans are good at checking things computers can not. In this final post in the series, we’re going to look at the grey area in betwee…