Code review just got even better in Bitbucket Cloud

By on April 6, 2016

High performing teams use code reviews. A lot. And most of the time code reviews are done using pull requests.

Improved pull request list
We’ve updated Bitbucket’s pull request list to make it even easier for you to quickly find the pull requests you need to review, and to show you the state of a review at a glance:

We’ve improved the filter bar to let you jump right to the pull request you need to approve and filter by author, target branch or pull request state. You can search for the pull request too. The updated display shows who’s reviewing each pull request, who has already approved it, and which branch it’s targeting, so it’s easy to see how your team’s code reviews are progressing.

Manage features
Try out the new pull request list today by visiting the Labs page and turning on the improved pull request list:

Please provide feedback straight from the Labs page by clicking “Give feedback”. Your feedback will help us get it right. We’ll be rolling out the improved pull request list to everyone over the new few weeks.

We will use the Labs functionality in Bitbucket Cloud to gradually roll-out new features to our users. This will let you provide extensive feedback to our engineering teams and help make the feature better before we roll it out to everyone.

Ignore whitespace
Remember when you had to add ?w=1 to your page URL to ignore whitespace? That did the trick, but it wasn’t very intuitive. Now it is.

Now, every time you’re reviewing some code and want to ignore whitespace on that diff, click the “More Actions” menu (i.e., the ellipses “…”) and select “Ignore whitespace”. Selecting this option will ignore whitespace across all changed files. This option is available for everyone today.

We hope these new features we introduced recently improve your code reviewing experience in Bitbucket Cloud. Happy coding!

 

Did you find this post useful? Share it on your social network of choice so your fellow Bitbucketeers can learn about these new goodies, too!

6 Comments

  • Robert Watkins
    Posted April 6, 2016 at 2:15 pm | Permalink

    Kudos for finally getting this in.

    It would be nice if the “Ignore Whitespace” was a global option that was remembered, instead of a URL parameter (even with a nicer UI for turning it on). In general, if you care about ignoring whitespace, that’s going to be your default preference.

    It would be nicer still if the diff could show (perhaps with an icon or badge in the view) that there _is_ ignored whitespace (that is, if you weren’t ignoring the whitespace, there would be a different output). When I’m doing reviews for code logic, I don’t care about whitespace – but after that, I review for code style, where I might care.

    • Ben Echols
      Posted April 6, 2016 at 4:35 pm | Permalink

      Hi Robert,

      Thanks for the thoughtful suggestions. Both are enhancements that we discussed and hope to add soon. Exposing the option at the file level was the most flexible initial implementation, but we’ll definitely be iterating.

  • Robert Watkins
    Posted April 6, 2016 at 4:52 pm | Permalink

    Would it be possible to get the new pull request features on the pull-request list for the dashboard? When dealing with multiple repositories, that’s the main place I work with pull requests from.

    • Ben Echols
      Posted April 7, 2016 at 11:09 am | Permalink

      Yes, that’s great idea, and it’s something else we’re working towards.

  • Raj Sarkar
    Posted April 12, 2016 at 8:55 am | Permalink

    Pull request list is already in Bitbucket Server.

    • Jerry Tromp
      Posted April 20, 2016 at 3:23 am | Permalink

      It doesn’t have automatic updates to pull requests for one. It seems so far behind on the cloud version.