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 |
Revision history for this message
| Loïc Gomez (kotodama) wrote | #1 |
To post a comment you must log in.
+1 to that.
Alternatively, LP could add direct links to changed file sources.