)]}'
{"docker/helm-repository/Dockerfile.j2":[{"author":{"_account_id":22582,"name":"Duong Ha-Quang","email":"quangduongha@gmail.com","username":"duonghq"},"change_message_id":"df72a102b7c555e1b1b70659bc4370e15a6b0ca1","unresolved":false,"context_lines":[{"line_number":51,"context_line":"{% elif base_arch \u003d\u003d \u0027aarch64\u0027 %}"},{"line_number":52,"context_line":"ENV helm_arch\u003darm64"},{"line_number":53,"context_line":"{% else %}"},{"line_number":54,"context_line":"ENV helm_arch\u003d{{ base_arch }}"},{"line_number":55,"context_line":"{% endif %}"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"{% block helm_repository_install_kubernetes_helm %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"df87a7cf_107a8987","line":54,"range":{"start_line":54,"start_character":14,"end_line":54,"end_character":29},"updated":"2017-12-14 08:11:44.000000000","message":"should be put in other ps with description.","commit_id":"3364720f3435a7e5469e0f1ed0bce4c02e70af9d"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"1817553ac99f9bb4d70613e474cb9ca3f6627181","unresolved":false,"context_lines":[{"line_number":51,"context_line":"{% elif base_arch \u003d\u003d \u0027aarch64\u0027 %}"},{"line_number":52,"context_line":"ENV helm_arch\u003darm64"},{"line_number":53,"context_line":"{% else %}"},{"line_number":54,"context_line":"ENV helm_arch\u003d{{ base_arch }}"},{"line_number":55,"context_line":"{% endif %}"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"{% block helm_repository_install_kubernetes_helm %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"df87a7cf_4b9aa06e","line":54,"range":{"start_line":54,"start_character":14,"end_line":54,"end_character":29},"in_reply_to":"df87a7cf_107a8987","updated":"2017-12-14 09:16:59.000000000","message":"I think the small logic cleanup is appropriate here, since I\u0027m touching that part anyway. Not a reason to block the patch IMO.","commit_id":"3364720f3435a7e5469e0f1ed0bce4c02e70af9d"},{"author":{"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"},"change_message_id":"68515d080621b9cb414a8e05de01d5ec6f1352a2","unresolved":false,"context_lines":[{"line_number":51,"context_line":"{% elif base_arch \u003d\u003d \u0027aarch64\u0027 %}"},{"line_number":52,"context_line":"ENV helm_arch\u003darm64"},{"line_number":53,"context_line":"{% else %}"},{"line_number":54,"context_line":"ENV helm_arch\u003d{{ base_arch }}"},{"line_number":55,"context_line":"{% endif %}"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"{% block helm_repository_install_kubernetes_helm %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"df87a7cf_797b468b","line":54,"range":{"start_line":54,"start_character":14,"end_line":54,"end_character":29},"in_reply_to":"df87a7cf_4b9aa06e","updated":"2017-12-19 15:42:15.000000000","message":"Logical cleanup to block of code that needed to be modified to pass new tests.","commit_id":"3364720f3435a7e5469e0f1ed0bce4c02e70af9d"}],"tools/validate-indentation.sh":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8fd25d02b1e073f56fe72bc19d2fdd4609fe7c8a","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"RES\u003d0"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"for dockerfile in \"$@\"; do"},{"line_number":6,"context_line":"    if grep -qE \u0027^\\s+[A-Z]+\\s\u0027 \"$dockerfile\"; then"},{"line_number":7,"context_line":"        echo \"ERROR: $dockerfile has indented Dockerfile instruction\" \u003e\u00262"},{"line_number":8,"context_line":"        RES\u003d1"}],"source_content_type":"text/x-sh","patch_set":1,"id":"df87a7cf_2c5076d4","line":5,"updated":"2017-12-05 12:53:29.000000000","message":"how about check the rendered files?","commit_id":"3364720f3435a7e5469e0f1ed0bce4c02e70af9d"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"5431d55cfd2a6b2938ebee0701ab52887f53dfdd","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"RES\u003d0"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"for dockerfile in \"$@\"; do"},{"line_number":6,"context_line":"    if grep -qE \u0027^\\s+[A-Z]+\\s\u0027 \"$dockerfile\"; then"},{"line_number":7,"context_line":"        echo \"ERROR: $dockerfile has indented Dockerfile instruction\" \u003e\u00262"},{"line_number":8,"context_line":"        RES\u003d1"}],"source_content_type":"text/x-sh","patch_set":1,"id":"df87a7cf_ac5fa670","line":5,"in_reply_to":"df87a7cf_2c5076d4","updated":"2017-12-05 13:06:11.000000000","message":"We can\u0027t because of conditionals in j2.","commit_id":"3364720f3435a7e5469e0f1ed0bce4c02e70af9d"}]}