1216078 - Mozreview should not post full patch description each time a patch gets updated

Status: RESOLVED FIXED

(MozReview Graveyard :: General, defect, P3)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

As you can see on bug 1211489 it can be very distracting if you update a patch multiple times which itself has a long description set. Instead of posting the full description each time, we may be able to only post the changelog? Right now its not obvious what has been changed exactly when simply following the bug comments.

Comment 1

[Dan Minor [:dminor]]

I believe we have plans in the works to reduce the amount of information we mirror to bugzilla in general, but there's no reason we can't make this better now.

Comment 2

[Dan Minor [:dminor]]

Attachment: MozReview Request: mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r=mcote

mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r?mcote

At the moment we create a comment containing the full commit description each
time a commit is updated. If people write detailed commit messages, this ends
up taking a large amount of space when looking at the bug.

With these changes, we only create a comment with the full commit description
on the first change. The comment for each subsequent update to a commit only
contains a link to the review.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

That means Mozreview currently doesn't support incremental comments? To follow the process of follow-up commits that would be kinda helpful.

Comment 4

[Dan Minor [:dminor]]

(In reply to Henrik Skupin (:whimboo) from comment #3)
> That means Mozreview currently doesn't support incremental comments? To
> follow the process of follow-up commits that would be kinda helpful.

I am trying to minimize the amount of information that is duplicated back to bugzilla.

I'm sorry, I'm not sure what additional information you're looking for. The list of affected files for the commit?

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to Dan Minor [:dminor] from comment #4)
> I'm sorry, I'm not sure what additional information you're looking for. The
> list of affected files for the commit?

Just the ability to specify some lines for changes since the last commit for this patch. So when you see the comment it is clear what has been changed since the last commit.

Comment 6

[Dan Minor [:dminor]]

(In reply to Henrik Skupin (:whimboo) from comment #5)
> (In reply to Dan Minor [:dminor] from comment #4)
> > I'm sorry, I'm not sure what additional information you're looking for. The
> > list of affected files for the commit?
> 
> Just the ability to specify some lines for changes since the last commit for
> this patch. So when you see the comment it is clear what has been changed
> since the last commit.

That makes sense but we don't have a direct way of capturing that information. My original thought was to look at the interdiff and list the files that have changed, but I think in general we're trying to limit the amount of review information that goes back to bugzilla.

Comment 7

[Gregory Szorc [:gps]]

Comment on attachment 8675862 [details]
MozReview Request: mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r=mcote

https://reviewboard.mozilla.org/r/22467/#review20139

::: pylib/rbbz/rbbz/extension.py:287
(Diff revision 1)
> +        comment = 'Review requested updated, see: %s\n\n' % get_obj_url(review_request)

This would be even cooler if we displayed a link to the interdiff.

Followup?

Comment 8

[Dan Minor [:dminor]]

https://reviewboard.mozilla.org/r/22467/#review20139

> This would be even cooler if we displayed a link to the interdiff.
> 
> Followup?

Good idea, I'll do that instead.

Comment 9

[Dan Minor [:dminor]]

Comment on attachment 8675862 [details]
MozReview Request: mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r=mcote

mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r?mcote

At the moment we create a comment containing the full commit description each
time a commit is updated. If people write detailed commit messages, this ends
up taking a large amount of space when looking at the bug.

With these changes, we only create a comment with the full commit description
on the first change. The comment for each subsequent update to a commit only
contains a link to the interdiff.

Comment 10

[Mark Côté [:mcote]]

Comment on attachment 8675862 [details]
MozReview Request: mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r=mcote

https://reviewboard.mozilla.org/r/22467/#review20153

::: pylib/rbbz/rbbz/extension.py:288
(Diff revision 2)
> +        comment = 'Review request updated, see: %sdiff/%d-%d/\n' % (

Semicolon, not comma.  Also maybe mention this is an interdiff link ("see interdiff" or "see changes in this revision" or the like).

::: pylib/rbbz/rbbz/extension.py:289
(Diff revision 2)
> +                  get_obj_url(review_request),
> +                  diffset_count,
> +                  diffset_count + 1)

Weird indentation.

Comment 11

[Dan Minor [:dminor]]

Comment on attachment 8675862 [details]
MozReview Request: mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r=mcote

mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r?mcote

At the moment we create a comment containing the full commit description each
time a commit is updated. If people write detailed commit messages, this ends
up taking a large amount of space when looking at the bug.

With these changes, we only create a comment with the full commit description
on the first change. The comment for each subsequent update to a commit only
contains a link to the interdiff.

Comment 12

[Mark Côté [:mcote]]

Comment on attachment 8675862 [details]
MozReview Request: mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r=mcote

https://reviewboard.mozilla.org/r/22467/#review20607

Comment 13

[Dan Minor [:dminor]]

Comment on attachment 8675862 [details]
MozReview Request: mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r=mcote

mozreview: only mirror full commit description to bugzilla when first publishing (bug 1216078) r=mcote

At the moment we create a comment containing the full commit description each
time a commit is updated. If people write detailed commit messages, this ends
up taking a large amount of space when looking at the bug.

With these changes, we only create a comment with the full commit description
on the first change. The comment for each subsequent update to a commit only
contains a link to the interdiff.

Comment 14

[Dan Minor [:dminor]]

Thanks!

https://hg.mozilla.org/hgcustom/version-control-tools/rev/608df44d9ff3