)]}'
{"playbooks/plugins/lookups/py_pkgs.py":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"3068b7d219077d7bc23e35012b53bfeb3cfa02e0","unresolved":false,"context_lines":[{"line_number":29,"context_line":"REQUIREMENTS_FILE_TYPES \u003d ["},{"line_number":30,"context_line":"    \u0027global-requirements.txt\u0027,"},{"line_number":31,"context_line":"    \u0027test-requirements.txt\u0027,"},{"line_number":32,"context_line":"    \u0027dev-requirements.txt\u0027,"},{"line_number":33,"context_line":"    \u0027requirements.txt\u0027,"},{"line_number":34,"context_line":"    \u0027global-requirement-pins.txt\u0027"},{"line_number":35,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":10,"id":"da85f559_1e5ebe76","line":32,"updated":"2015-11-13 09:04:58.000000000","message":"Should this be left as test-requirements as we\u0027re renaming this file?","commit_id":"c6968f6f09d2fda39deba6d0038de67986efb440"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"d7cff8b91e741585cf2527f991613f5003fddb31","unresolved":false,"context_lines":[{"line_number":29,"context_line":"REQUIREMENTS_FILE_TYPES \u003d ["},{"line_number":30,"context_line":"    \u0027global-requirements.txt\u0027,"},{"line_number":31,"context_line":"    \u0027test-requirements.txt\u0027,"},{"line_number":32,"context_line":"    \u0027dev-requirements.txt\u0027,"},{"line_number":33,"context_line":"    \u0027requirements.txt\u0027,"},{"line_number":34,"context_line":"    \u0027global-requirement-pins.txt\u0027"},{"line_number":35,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":10,"id":"da85f559_cc671ee9","line":32,"in_reply_to":"da85f559_1e5ebe76","updated":"2015-11-13 14:24:12.000000000","message":"No I think it should remain for now because it doesn\u0027t hurt the application.","commit_id":"c6968f6f09d2fda39deba6d0038de67986efb440"},{"author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"change_message_id":"c8def587dbc720012a8a9aaee1fa8f3041b05487","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    else:"},{"line_number":80,"context_line":"        _git_url \u003d _git_url[0]"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    git_branch_sha \u003d _git_url.split(\u0027@\u0027)"},{"line_number":83,"context_line":"    if len(git_branch_sha) \u003e 1:"},{"line_number":84,"context_line":"        url, branch \u003d git_branch_sha"},{"line_number":85,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":10,"id":"da85f559_82b57a32","line":82,"updated":"2015-11-13 03:20:41.000000000","message":"This can\u0027t parse https://authtoken@github.com/organization/privaterepo\n\nMaybe something more like...\n-    if len(git_branch_sha) \u003e 1:\n+    if len(git_branch_sha) \u003e 2:\n+        branch \u003d git_branch_sha.pop()\n+        url \u003d \u0027@\u0027.join(git_branch_sha)\n+    elif len(git_branch_sha) \u003e 1:","commit_id":"c6968f6f09d2fda39deba6d0038de67986efb440"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"d7cff8b91e741585cf2527f991613f5003fddb31","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    else:"},{"line_number":80,"context_line":"        _git_url \u003d _git_url[0]"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    git_branch_sha \u003d _git_url.split(\u0027@\u0027)"},{"line_number":83,"context_line":"    if len(git_branch_sha) \u003e 1:"},{"line_number":84,"context_line":"        url, branch \u003d git_branch_sha"},{"line_number":85,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":10,"id":"da85f559_6c9f12b5","line":82,"in_reply_to":"da85f559_82b57a32","updated":"2015-11-13 14:24:12.000000000","message":"done","commit_id":"c6968f6f09d2fda39deba6d0038de67986efb440"}],"playbooks/roles/repo_build/defaults/main.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"3068b7d219077d7bc23e35012b53bfeb3cfa02e0","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# This is disabled in Kilo due to the upper-constraints file not be fully backed in kilo"},{"line_number":17,"context_line":"repo_build_use_upper_constraints: false "},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"repo_build_service_user_name: \"nginx\""},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"da85f559_5ef20671","line":17,"updated":"2015-11-13 09:04:58.000000000","message":"nit: whitespace\nAnd yes, as per the release team the upper constraints file is not being maintained for Kilo. I\u0027ll try and have a chat with them today to determine the best strategy for this.","commit_id":"c6968f6f09d2fda39deba6d0038de67986efb440"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"d7cff8b91e741585cf2527f991613f5003fddb31","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# This is disabled in Kilo due to the upper-constraints file not be fully backed in kilo"},{"line_number":17,"context_line":"repo_build_use_upper_constraints: false "},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"repo_build_service_user_name: \"nginx\""},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"da85f559_ec92229f","line":17,"in_reply_to":"da85f559_5ef20671","updated":"2015-11-13 14:24:12.000000000","message":"Done","commit_id":"c6968f6f09d2fda39deba6d0038de67986efb440"}]}