ltoscano added a comment.

  For the people that will land up here later: it seems that both are valid 
(BUG and BUGS), see line 422 here:
  
https://websvn.kde.org/trunk/kde-common/svn/hooks/post-commit.pl?revision=1446260&view=markup
  
  About the reviewers, you don't need to set the reviewer if the automated 
Herald rule sets a mailing list to the subscribers list. The mail will land in 
the proper place, despite what Phabricator says.
  And even if having a reviewer is preferred, then the rules should be updated 
rather than adding explicit reviewers IMHO.

REPOSITORY
  R223 Okular

REVISION DETAIL
  https://phabricator.kde.org/D7874

To: aacid, rkflx
Cc: ltoscano, ngraham, rkflx, tobiasdeiminger, #okular, aacid

Reply via email to