)]}'
{
  "commit": "a2e5ab82c760e1010dc748397d972350f92d5e8e",
  "tree": "48f4859a87958f6b809cd1025d333ff579edf03a",
  "parents": [
    "a2e152a139cf1469ac43ee6d32c468d9286d1837"
  ],
  "author": {
    "name": "Mårten Kongstad",
    "email": "amhk@google.com",
    "time": "Mon Jun 19 16:28:54 2023 +0200"
  },
  "committer": {
    "name": "Mårten Kongstad",
    "email": "amhk@google.com",
    "time": "Mon Jun 19 16:53:22 2023 +0200"
  },
  "message": "aconfig: make proto fields optional\n\nChange all required proto fields to optional. While the proto file is\nsupposed to be a backwards compatible API, and fields are not supposed\nto be deprecated, this commit will allow for that option if needed.\n\nImplementation wise this change doesn\u0027t matter much: any parsed data\nneeds additional verification outside what the protobuf crate\u0027s parser\nprovides anyway, so adding checks to verify that all required fields,\neven though marked optional in the proto file, were found is a minor\nincrease in code complexity.\n\nIf in the future a proto field should no longer be used:\n\n  - keep the field in the proto, still marked optional and clearly\n    document that it is no longer in use\n  - change protos.rs from checking struct.has_field() to explicitly\n    dropping any value via struct.clear_field()\n\nBug: 286337317\nTest: atest aconfig.test\nChange-Id: Iad1ccfe50ecac286ff7a796aec909bec70b9520d\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "9f6424f440aeefb23a347cc43a5bdbaeff9b1039",
      "old_mode": 33188,
      "old_path": "tools/aconfig/protos/aconfig.proto",
      "new_id": "b59fdfc61669992820f04281244efdaa692fec11",
      "new_mode": 33188,
      "new_path": "tools/aconfig/protos/aconfig.proto"
    },
    {
      "type": "modify",
      "old_id": "fb5dab4fe23772d907cc0ef632a0d9678a0f1bf8",
      "old_mode": 33188,
      "old_path": "tools/aconfig/src/protos.rs",
      "new_id": "beebd9341be8dfeec0bdf0da99c6f79f9773102e",
      "new_mode": 33188,
      "new_path": "tools/aconfig/src/protos.rs"
    }
  ]
}