)]}'
{"docker/marathon/Dockerfile.j2":[{"author":{"_account_id":16620,"name":"Alicja Kwasniewska","email":"alicja.kwasniewska@intel.com","username":"akwasnie"},"change_message_id":"70d085b4130bb2461263b4f393bc23c778d54d74","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% elif base_distro in [\u0027ubuntu\u0027, \u0027debian\u0027] %}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":13,"context_line":"COPY sources.list.{{ base_distro }} /etc/apt/sources.list.d/openjdk.list"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN apt-key adv --keyserver keyserver.ubuntu.com --recv DA1A4A13543B466853BAF164EB9B1D8886F44E2A \\"},{"line_number":16,"context_line":"    \u0026\u0026 apt-get update"},{"line_number":17,"context_line":"{% endif %}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# NOTE(nihilifer): Marathon package for Ubuntu depends on Oracle Java and"},{"line_number":20,"context_line":"# there is no official way to use OpenJDK. There is no way to accept the"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"da0c15f0_e67ca274","line":17,"range":{"start_line":12,"start_character":0,"end_line":17,"end_character":11},"updated":"2016-03-25 15:55:28.000000000","message":"could you add indent here?","commit_id":"02ce6adc8fa9e79a93d4a30620a5902608172dd3"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"c813b235a3931d772dbd2b938a6fe323594d2338","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% elif base_distro in [\u0027ubuntu\u0027, \u0027debian\u0027] %}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"{% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":13,"context_line":"COPY sources.list.{{ base_distro }} /etc/apt/sources.list.d/openjdk.list"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN apt-key adv --keyserver keyserver.ubuntu.com --recv DA1A4A13543B466853BAF164EB9B1D8886F44E2A \\"},{"line_number":16,"context_line":"    \u0026\u0026 apt-get update"},{"line_number":17,"context_line":"{% endif %}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# NOTE(nihilifer): Marathon package for Ubuntu depends on Oracle Java and"},{"line_number":20,"context_line":"# there is no official way to use OpenJDK. There is no way to accept the"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"da0c15f0_cc5d3deb","line":17,"range":{"start_line":12,"start_character":0,"end_line":17,"end_character":11},"in_reply_to":"da0c15f0_e67ca274","updated":"2016-03-25 16:36:10.000000000","message":"do you mean ident the jinja2 statements ? \nthats done. I thing the COPY/RUN statement cannot indented.","commit_id":"02ce6adc8fa9e79a93d4a30620a5902608172dd3"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"8353e79908a3864e3d34cd074db162f7c73c36b3","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    {% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"COPY sources.list.{{ base_distro }} /etc/apt/sources.list.d/openjdk.list"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"RUN apt-key adv --keyserver keyserver.ubuntu.com --recv DA1A4A13543B466853BAF164EB9B1D8886F44E2A \\"},{"line_number":17,"context_line":"    \u0026\u0026 apt-get update"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba0121b8_87f4ad87","line":14,"updated":"2016-03-29 19:36:03.000000000","message":"I would think it\u0027d be clearer to just spell out the base_distro since at this point it has already been defined.","commit_id":"815e31bb7fe6ea94aeb6a916a7d203d19359e3f4"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"a8d3a4c7852c50dc2006ea8e735c45c11138f48b","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    {% if base_distro \u003d\u003d \u0027ubuntu\u0027 %}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"COPY sources.list.{{ base_distro }} /etc/apt/sources.list.d/openjdk.list"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"RUN apt-key adv --keyserver keyserver.ubuntu.com --recv DA1A4A13543B466853BAF164EB9B1D8886F44E2A \\"},{"line_number":17,"context_line":"    \u0026\u0026 apt-get update"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"ba0121b8_68e836e3","line":14,"in_reply_to":"ba0121b8_87f4ad87","updated":"2016-03-30 07:32:50.000000000","message":"I better like it this way, since it\u0027s consistent with other\nplaces where sources lists are included.\nIf other reviewers think this needs changing I can do so. But for know I prefer to not change","commit_id":"815e31bb7fe6ea94aeb6a916a7d203d19359e3f4"}]}