)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"1debbbf36744e420a2531270099e54d092c11ebb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1770ad6d_eeb05015","updated":"2023-06-26 11:31:19.000000000","message":"Feel free to +w at your own discretion about the line length Alex mentioned.","commit_id":"402d63903e71211ceefb08f5795ac7a7120a0f8f"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"cdba8eb810a8d1ecd3c8581b8fa0b4a25b636fbb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"685e9f4b_309c6dd1","updated":"2023-06-22 09:31:50.000000000","message":"doesn\u0027t make sense that we test Puppet logic for type validation now that we change to that, so don\u0027t need to test and catch Puppet::PreformattedError right?","commit_id":"402d63903e71211ceefb08f5795ac7a7120a0f8f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4a62fef1817da7705c12f2af3079021ef449c77f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1d348834_bf9f2825","in_reply_to":"685e9f4b_309c6dd1","updated":"2023-06-22 09:50:50.000000000","message":"yeah I think we can remove the test case now. Alternatively we can keep it without message assertions but having test cases for individual type check does not make much sense to me.","commit_id":"402d63903e71211ceefb08f5795ac7a7120a0f8f"}],"manifests/cron/purge_deleted.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"6a91234f5c74d67aeab4e99a08ee431e27d5ea64","unresolved":false,"context_lines":[{"line_number":61,"context_line":"  $user                             \u003d $::heat::params::user,"},{"line_number":62,"context_line":"  $age                              \u003d 1,"},{"line_number":63,"context_line":"  Enum[\u0027days\u0027, \u0027hours\u0027, \u0027minutes\u0027, \u0027seconds\u0027] $age_type"},{"line_number":64,"context_line":"    \u003d \u0027days\u0027,"},{"line_number":65,"context_line":"  $destination                      \u003d \u0027/var/log/heat/heat-purge_deleted.log\u0027,"},{"line_number":66,"context_line":"  $batch_size                       \u003d undef,"},{"line_number":67,"context_line":") inherits heat::params {"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"06f914ef_8f80528d","line":64,"updated":"2023-06-22 14:33:28.000000000","message":"I\u0027m assuming the line break here is to meet line length issues? I\u0027d kinda prefer just ignoring line length for this rather than splitting it.","commit_id":"402d63903e71211ceefb08f5795ac7a7120a0f8f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2bacc2b8caba9aea6bb7359e93361c7766c03db1","unresolved":false,"context_lines":[{"line_number":61,"context_line":"  $user                             \u003d $::heat::params::user,"},{"line_number":62,"context_line":"  $age                              \u003d 1,"},{"line_number":63,"context_line":"  Enum[\u0027days\u0027, \u0027hours\u0027, \u0027minutes\u0027, \u0027seconds\u0027] $age_type"},{"line_number":64,"context_line":"    \u003d \u0027days\u0027,"},{"line_number":65,"context_line":"  $destination                      \u003d \u0027/var/log/heat/heat-purge_deleted.log\u0027,"},{"line_number":66,"context_line":"  $batch_size                       \u003d undef,"},{"line_number":67,"context_line":") inherits heat::params {"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"2f018969_a51f6c34","line":64,"in_reply_to":"06f914ef_8f80528d","updated":"2023-06-26 13:09:48.000000000","message":"yeah though we can probably just align \u003d as we do usually. I\u0027ve updated the change to avoid this line break (and align \u003d)","commit_id":"402d63903e71211ceefb08f5795ac7a7120a0f8f"}]}