)]}'
{
  "commit": "364152186c6258725b5e5ce2a93be55c2a911133",
  "tree": "54b9b5e0613b3728b5691ab47048e830dcdc1fb4",
  "parents": [
    "b56e764ce200c2bd33ebdd1b557982148ab8b68e"
  ],
  "author": {
    "name": "Edwin Kempin",
    "email": "ekempin@google.com",
    "time": "Fri Nov 20 11:01:39 2020 +0100"
  },
  "committer": {
    "name": "Edwin Kempin",
    "email": "ekempin@google.com",
    "time": "Fri Nov 20 11:31:49 2020 +0100"
  },
  "message": "AbstractGetCodeOwnersForPath: Replace suggest flag with overridable method\n\nInstead of having a boolean suggest flag that is true for\nGetCodeOwnersForPathInChange and false for GetCodeOwnersForPathInBranch\nadd a filterCodeOwners that can be overridden by the subclasses to\nfilter out additional results. Use this method in\nGetCodeOwnersForPathInChange to filter out the service users that should\nnot show up in the code owner suggestion. Since the methods to detect\nand filter out service users are only needed in\nGetCodeOwnersForPathInChange they are moved into this class.\n\nThe new filterCodeOwners method has a parameter to provide the resource\non which the request is being performed. For this parameter we want to\nuse the resource type that is used by the subclass. To make this\npossible we add a generic type for the resource to\nAbstractGetCodeOwnersForPath.\n\nSince we are already touching the filterOutServiceUsers method, add some\ncomments to explain the return values to avoid misreading them.\n\nSigned-off-by: Edwin Kempin \u003cekempin@google.com\u003e\nChange-Id: Ia7bbf579e808af4f3d615aa6edce1b24006c4afe\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "e3c7a1f875c954d609d04241c62a742475d5aaf7",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/plugins/codeowners/restapi/AbstractGetCodeOwnersForPath.java",
      "new_id": "a5f7b5d7151df93ef02c19f6a23b1adf04670242",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/plugins/codeowners/restapi/AbstractGetCodeOwnersForPath.java"
    },
    {
      "type": "modify",
      "old_id": "a94d84877fbc4b5e22c844cb13152df01daaa05a",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/plugins/codeowners/restapi/GetCodeOwnersForPathInBranch.java",
      "new_id": "3fab53744ef16f1cfa0511b23cc693af03166f5d",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/plugins/codeowners/restapi/GetCodeOwnersForPathInBranch.java"
    },
    {
      "type": "modify",
      "old_id": "216d2644bf25ac923664e67e0446c5b3a59b4b4c",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/plugins/codeowners/restapi/GetCodeOwnersForPathInChange.java",
      "new_id": "05951778fd19617a50d38505422b222d5bf7d73d",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/plugins/codeowners/restapi/GetCodeOwnersForPathInChange.java"
    }
  ]
}