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/EjwD7Pi7J_0 (more…)

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, available in Java 10 and 11. If your application is using any version later than Java 8 you may benefit fr

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 you have a branch review, it'll automatically add new revisions to it. It'll promptly notify you abou

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 workflow, whatever it looks like, simpler by automating as much of it as possible. We'll show the types

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. This third approach for code reviews puts learning at the centre of the process — the purpose is not to

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 checked frequently while it's being built, so design feedback comes at the right time. In this exa

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 example we're going to explore is Code Review as a Gateway. In this process, a developer picks up or i

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'd like to share these and more in the form of feedback you could give during a review of Java 8 cod

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 are teams that want to make sure all commits are reviewed by at least three team members. Some team

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 barely scratched the surface of it, but still these posts proved to be interesting and helpful to many

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 between - we’re going to look at how the features in Upsource can make your job as a human code reviewer e

This is part 6 of 6 posts on what to look for in a code review. See previous posts from the series. How much work you do building a secure, robust system is like anything else on your project - it depends upon the project itself, where it’s running, who’s using it, what data it has access to, etc. Often, if our team doesn't have access to security experts, we go too far in one direction or the