Feature request: show entire file in diffs generated for merge requests
Bug #1392282 reported by
Olivier Tilloy
This bug affects 5 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Triaged
|
Low
|
Unassigned |
Bug Description
When reviewing code changes in a merge request, it would be very useful to have the ability to show the entire contents of changed files, not just the surrounding 3 lines for each changeset. This could be off by default, with an option to turn it on at will from within the diff itself.
As things stand today, I always end up grabbing a local copy of the target branch, merge the source branch into it, and use "bzr qdiff" with its "View Options > Complete" setting to do a proper code review. It would be great if I didn’t have to leave my browser tab to do that.
tags: | added: code-review ui |
Changed in launchpad: | |
status: | New → Triaged |
importance: | Undecided → Low |
tags: | added: feature |
To post a comment you must log in.
+1 to that.
Alternatively, LP could add direct links to changed file sources.