1087658 - Added bug comments when adjusting reviewers doesn't make a lot of sense

Status: RESOLVED FIXED

(MozReview Graveyard :: General, defect, P3)

Description

[Gregory Szorc [:gps]]

As part of writing tests to cover the review flag integration, I've noticed the Review Board likes to add comments every time it adjusts review flags. This can result in a redundant mass of similar comments. Take this test excerpt for example:

---

Removing all reviewers should remove all flags

  $ rbmanage remove-reviewer $HGPORT1 3 --user reviewer
  0 people listed on review request

  $ rbmanage publish $HGPORT1 3

  $ bugzilla dump-bug 2
  Bug 2:
    attachments:
    - attacher: [e-mail]
      content_type: text/x-review-board-request
      data: http://example.com/r/3/
      description: 'MozReview Request: bz://2/mynick'
      flags: []
      id: 2
      summary: 'MozReview Request: bz://2/mynick'
    comments:
    - author: [e-mail]
      id: 6
      tags: []
      text: ''
    - author: [e-mail]
      id: 7
      tags: []
      text: 'Created attachment 2 [details] [diff] [review]
  
        MozReview Request: bz://2/mynick'
    - author: [e-mail]
      id: 8
      tags: []
      text: '/r/4 - Bug 2 - Multiple reviewers
  
  
        Pull down this commit:
  
  
        hg pull review -r d17099d7ee43e288f43e0210edc71b9782f92b77'
    - author: [e-mail]
      id: 9
      tags: []
      text: '/r/4 - Bug 2 - Multiple reviewers
  
  
        Pull down this commit:
  
  
        hg pull review -r d17099d7ee43e288f43e0210edc71b9782f92b77'
    - author: [e-mail]
      id: 10
      tags: []
      text: '/r/4 - Bug 2 - Multiple reviewers
  
  
        Pull down this commit:
  
  
        hg pull review -r d17099d7ee43e288f43e0210edc71b9782f92b77'
    summary: Multiple Reviewers

---

We should make the comment more intelligent - possibly by removing the text completely from flag-only adjustments.

Comment 2

[Mark Côté [:mcote]]

We have too many P1s, so I'm spreading out the priorities.  P3 -> P4, P2 -> P3, and some portion of P1s will become P2.

Comment 3

[Dan Minor [:dminor]]

I believe this was fixed as part of Bug 1229789.