It would be nice if the diffs sent by launchpad could be generated with --diff-options=-p
Bug #904204 reported by
James Troup
This bug report is a duplicate of:
Bug #608114: Show modified class/method names in the diff of the merge proposals.
Edit
Remove
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Triaged
|
Low
|
Unassigned |
Bug Description
It'd be nice if the bzr diffs sent in email by Launchpad could be
generated with --diff-options=-p.
I don't know if it's viable to change globally, if not, it's probably
not worth the complexity of per branch/
options.
Changed in launchpad: | |
status: | New → Triaged |
importance: | Undecided → Low |
To post a comment you must log in.
This would also be really nice in the diffs shown in merge requests. I think there's no context where it's better to not use "diff -p", usability-wise. At present the diffs are essentially useless for reviewing code because even good changes to code may need a r- if they cause a function name to become no-longer-correct, so one has to check out the branch and run bzr diff manually.