)]}'
{
  "commit": "b33d21bdaf4f33338be89c11cd9a641ebd7edcd0",
  "tree": "fcac89354d353db5504b787c3160f5dfd4f72505",
  "parents": [
    "abc9a647a2c165e021427b70fcbca8e16cce9b2f"
  ],
  "author": {
    "name": "Paul Duffin",
    "email": "paulduffin@google.com",
    "time": "Fri May 01 18:13:36 2020 +0100"
  },
  "committer": {
    "name": "Paul Duffin",
    "email": "paulduffin@google.com",
    "time": "Tue May 05 10:34:55 2020 +0100"
  },
  "message": "Report visibility errors in both check and gather phases\n\nPreviously, errors that were found when splitting visibility rules were\nonly reported in the check phase and simply ignored during the gather\nphase. That was because every visibility property that was processed in\nthe gather phase had already been checked in the check phase. However,\nthat is not strictly true as it has always been possible to add a\nmutator between the check and gather phases that creates a module with\ninvalid visibility properties that will just be ignored. Fortunately,\nthat has not happened.\n\nA follow up commit will add the capability to create modules after the\ndefaults have been applied which means the chances of invalid visibility\nproperties being ignored will increase.\n\nThis change makes both phases report any errors they find. It will not\nhave any impact on existing code as if any errors are reported in the\ncheck phase then the build will exit before the gather phase. It will\nprevent any invalid visibility errors from being ignored.\n\nBug: 155295806\nTest: m nothing\nMerged-In: I101fa6b03d2530b16e4394a9e466fead48be0ff0\nChange-Id: I101fa6b03d2530b16e4394a9e466fead48be0ff0\n(cherry picked from commit 0c83aba28ef5862aed2ce7f5beb649c3677d9008)\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "ab164446576a9ac5d4d41856867d0cbaad95b9a5",
      "old_mode": 33188,
      "old_path": "android/visibility.go",
      "new_id": "1e3b91ddd56a61439179a8036c764f96759d0dd2",
      "new_mode": 33188,
      "new_path": "android/visibility.go"
    }
  ]
}