commit 0de334fe527a3825bd81a0a47c537a14d5889251
author Edwin Kempin <ekempin@google.com> Wed May 05 10:17:50 2021 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed May 05 10:17:50 2021 +0000
tree 8172915baf4b611e163b4b21d7d625b7f1a70d8c
parent da69415ae4578ce05c33c2fa1e5a7c74a70daa77
parent a61bb6e940ce00f2602d19163e8d410c857a649a

Merge "Apply implicit approvals only when change owner == last patch set uploader"

java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerSubmitRule.java

diff --git a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerSubmitRule.java b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerSubmitRule.java
index e69616d..df96cc4 100644
--- a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerSubmitRule.java
+++ b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerSubmitRule.java
@@ -146,9 +146,10 @@
         }
       }
       errorMessage += ".";
-      codeOwnerMetrics.countCodeOwnerSubmitRuleErrors.increment(cause);
 
       if (isRuleError) {
+        codeOwnerMetrics.countCodeOwnerSubmitRuleErrors.increment(cause);
+
         logger.atWarning().log(errorMessage);
         return Optional.of(ruleError(errorMessage));
       }