<table><tr><td style="">ltoscano added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7874" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D7874#150708" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D7874#150708</a>, <a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@rkflx</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>For me it's about signalling what the submitter want's to be done with the review:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">no reviewer: no review wanted, it's more like a WIP</li>
<li class="remarkup-list-item">group reviewer (could automatically default to <a href="https://phabricator.kde.org/tag/okular/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">#okular</a>): everybody interested should review</li>
<li class="remarkup-list-item">explicit reviewer: special expertise wanted</li>
</ul></div>
</blockquote>

<p>Please note that I was not talking specifically about Okular, but about the way the Herald rules are set up globally here on phabricator.kde.org.<br />
So your expectation does not match the general expectation. What I wrote is valid for Frameworks, for Plasma, etc.<br />
The expectation is that you don't need to add a reviewer if the automatic rule adds as *subscriber* a mailing list (where the review is delivered) or a project (which people can join and receive the notification).<br />
If you want a special person, sure, you can add that person as reviewer, but again what you expect for "group reviewer" is already fulfilled by groups in the subscriber field.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Anyway, it's not for me to interfere with the inner workings of the Okular project, this was merely a suggestion with only the best intentions in mind.</p></blockquote>

<p>Again, consider that the above indication is for all phabricator.kde.org.<br />
If you want it to be more in the way you think it should (so using explicit reviewers always), then the automatic rules which add subscribers should be changed (and you can propose this for example on kde-community@, opening a sysadming ticket for reference), but otherwise please follow the general expectation.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7874" rel="noreferrer">https://phabricator.kde.org/D7874</a></div></div><br /><div><strong>To: </strong>aacid, rkflx<br /><strong>Cc: </strong>ltoscano, ngraham, rkflx, tobiasdeiminger, Okular, aacid<br /></div>