Merge proposal page hides reviewable changes by including merges in diffs

Bug #1078026 reported by Aaron Bentley
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

When new revisions are added after an initial proposal, a diff is made available, and that diff combines all revisions made between each comment. But some of those revisions may be merges from trunk, which means that they do not need review. The changes introduced by the merge are essentially noise, making it hard to see the changes that do need review.

A simple solution would be to combine only non-merge revisions together.

Tags: code-review
Aaron Bentley (abentley)
tags: added: code-review
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.