)]}'
{"manifests/policy/base.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"866d2114ec4d1cc985eb7779bb34428d3464c572","unresolved":false,"context_lines":[{"line_number":29,"context_line":"#    Defaults to undef"},{"line_number":30,"context_line":"#"},{"line_number":31,"context_line":"#  [*file_format*]"},{"line_number":32,"context_line":"#    (optional) Format for file contents. Valid values "},{"line_number":33,"context_line":"#    are \u0027json\u0027 or \u0027yaml\u0027."},{"line_number":34,"context_line":"#    Defaults to \u0027json\u0027."},{"line_number":35,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"bf51134e_ffd9c582","line":32,"updated":"2020-07-13 01:41:56.000000000","message":"trailing space","commit_id":"fc1adfff1861c0332aa4d4162d451b31fa780be2"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"388d9dce7465f8a8c7b09b0343598d263089166e","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  $file_mode   \u003d \u00270640\u0027,"},{"line_number":41,"context_line":"  $file_user   \u003d undef,"},{"line_number":42,"context_line":"  $file_group  \u003d undef,"},{"line_number":43,"context_line":"  $file_format \u003d \u0027json\u0027,"},{"line_number":44,"context_line":") {"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"  ensure_resource(\u0027file\u0027, $file_path, {"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"bf51134e_ad2712c7","line":43,"range":{"start_line":43,"start_character":2,"end_line":43,"end_character":24},"updated":"2020-07-14 09:41:00.000000000","message":"Does it make sense to pick up file format from file_path like I did in my patch ?\n https://review.opendev.org/#/c/739989/7/manifests/policy/base.pp\n\nI think this is useful to avoid inconsistencies between file_path and file_format.","commit_id":"462270987992b9780e2a2413f40a51d38b06b5d9"}]}