)]}'
{"integration-test/test.sh":[{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"e237e42a5517f132aecc4945095f82f8bbfd1e2a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"# path-to-repo is an optional parameter, if it exists"},{"line_number":5,"context_line":"# no cloning will happen and the local directory will be used,"},{"line_number":6,"context_line":"# the first two parameter get ignored."},{"line_number":7,"context_line":"# Note: you can clone from a local file with REPO_ROOT\u003dfile:////~/path/to/repo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"set -x"},{"line_number":10,"context_line":"set -e"}],"source_content_type":"text/x-sh","patch_set":3,"id":"da6ed579_afb75df4","line":7,"updated":"2016-01-15 20:16:03.000000000","message":"So we already have REPO_ROOT for handling this, do we maybe want to also remove this logic altogether as redundant? Or instead of checking for file:// we should just unconditionally clone from that (if we haven\u0027t specified the third-argument).","commit_id":"40308d721470ee1d759bab63b9b9616e4557f044"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"09bf0e0dd04d2080cd55f802af4ba26de0a702e8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"# path-to-repo is an optional parameter, if it exists"},{"line_number":5,"context_line":"# no cloning will happen and the local directory will be used,"},{"line_number":6,"context_line":"# the first two parameter get ignored."},{"line_number":7,"context_line":"# Note: you can clone from a local file with REPO_ROOT\u003dfile:////~/path/to/repo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"set -x"},{"line_number":10,"context_line":"set -e"}],"source_content_type":"text/x-sh","patch_set":3,"id":"da6ed579_0f9229f7","line":7,"in_reply_to":"da6ed579_afb75df4","updated":"2016-01-15 20:28:44.000000000","message":"Let\u0027s remove the file:// special handling...","commit_id":"40308d721470ee1d759bab63b9b9616e4557f044"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"e237e42a5517f132aecc4945095f82f8bbfd1e2a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"REPO_ROOT\u003d${REPO_ROOT:-git://git.openstack.org}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"if [[ -z \"$2\" ]]; then"},{"line_number":15,"context_line":"    org\u003dopenstack"},{"line_number":16,"context_line":"    project\u003dnova"},{"line_number":17,"context_line":"else"},{"line_number":18,"context_line":"    org\u003d$1"}],"source_content_type":"text/x-sh","patch_set":3,"id":"da6ed579_4fcd8185","line":15,"updated":"2016-01-15 20:16:03.000000000","message":"I\u0027m also uncertain we actually need this check and code-path. We will probably always just use\n\n    tox -e integration {org} {repo} [{path}]\n\nSo we probably don\u0027t need to care about this case.","commit_id":"40308d721470ee1d759bab63b9b9616e4557f044"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"09bf0e0dd04d2080cd55f802af4ba26de0a702e8","unresolved":false,"context_lines":[{"line_number":12,"context_line":"REPO_ROOT\u003d${REPO_ROOT:-git://git.openstack.org}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"if [[ -z \"$2\" ]]; then"},{"line_number":15,"context_line":"    org\u003dopenstack"},{"line_number":16,"context_line":"    project\u003dnova"},{"line_number":17,"context_line":"else"},{"line_number":18,"context_line":"    org\u003d$1"}],"source_content_type":"text/x-sh","patch_set":3,"id":"da6ed579_4f76e1d9","line":15,"in_reply_to":"da6ed579_4fcd8185","updated":"2016-01-15 20:28:44.000000000","message":"Done","commit_id":"40308d721470ee1d759bab63b9b9616e4557f044"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"e237e42a5517f132aecc4945095f82f8bbfd1e2a","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    tempdir\u003d\"$(mktemp -d)\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    trap \"rm -rf $tempdir\" EXIT"},{"line_number":34,"context_line":"    pushd $tempdir"},{"line_number":35,"context_line":"    if [[ $REPO_ROOT  \u003d\u003d file://* ]]; then"},{"line_number":36,"context_line":"        git clone $REPO_ROOT/$org/$project"}],"source_content_type":"text/x-sh","patch_set":3,"id":"da6ed579_6fcac570","line":33,"updated":"2016-01-15 20:16:03.000000000","message":"This is a clever trick. Well done.","commit_id":"40308d721470ee1d759bab63b9b9616e4557f044"}]}