)]}'
{"manifests/redis.pp":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2161c8932b04265d3ebfe832a3138f300caf197a","unresolved":true,"context_lines":[{"line_number":12,"context_line":"    \u0027RedHat\u0027: {"},{"line_number":13,"context_line":"      $redis_package_name \u003d \u0027redis\u0027"},{"line_number":14,"context_line":"      $redis_service_name \u003d \u0027redis\u0027"},{"line_number":15,"context_line":"      if $::operatingsystemmajrelease \u003d\u003d \u00278\u0027 {"},{"line_number":16,"context_line":"        $redis_config       \u003d \u0027/etc/redis.conf\u0027"},{"line_number":17,"context_line":"      } else {"},{"line_number":18,"context_line":"        $redis_config       \u003d \u0027/etc/redis/redis.conf\u0027"},{"line_number":19,"context_line":"      }"},{"line_number":20,"context_line":"    }"},{"line_number":21,"context_line":"    default: {"},{"line_number":22,"context_line":"      fail(\"redis.pp manifest does not support family: ${::osfamily}\")"},{"line_number":23,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"9c17aa65_4a92cdad","line":20,"range":{"start_line":15,"start_character":0,"end_line":20,"end_character":5},"updated":"2021-05-20 11:39:21.000000000","message":"Can we check version \u003e 8 or not like ;\n\n```\nif versioncmp($::operatingsystemmajrelease, \u00278\u0027) \u003e 0 {\n  $redis_config       \u003d \u0027/etc/redis/redis.conf\u0027\n} else {\n  $redis_config       \u003d \u0027/etc/redis.conf\u0027\n}\n```\n\nI think this allows us to understand the intention of this code more clearly.","commit_id":"4a217c10d91f75ac233b581f8ad5c17fc973e9c1"}]}
