)]}'
{"functions-common":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"3e2421702d0ffc59082940c6208f91d85fcc061f","unresolved":false,"context_lines":[{"line_number":576,"context_line":"    GIT_FLAGS\u003d\"\""},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"    if [[ \"$GIT_DEPTH\" ]]; then"},{"line_number":579,"context_line":"        GIT_FLAGS\u003d\"$GIT_FLAGS --depth $GIT_DEPTH\""},{"line_number":580,"context_line":"    fi"},{"line_number":581,"context_line":""},{"line_number":582,"context_line":"    if [[ \"$OFFLINE\" \u003d \"True\" ]]; then"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"9aa7fdbe_28d1b6fb","line":579,"updated":"2014-10-20 21:12:33.000000000","message":"why not local?","commit_id":"e54929aeb6a2a0598ad594ba9d999cd384b23d1a"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"b1585285d737d7c1611df6c14bd2cd481b9f495f","unresolved":false,"context_lines":[{"line_number":575,"context_line":""},{"line_number":576,"context_line":"    RECLONE\u003d$(trueorfalse False $RECLONE)"},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"    if [[ \"$GIT_DEPTH\" ]]; then"},{"line_number":579,"context_line":"        git_clone_flags\u003d\"$git_clone_flags --depth $GIT_DEPTH\""},{"line_number":580,"context_line":"    fi"},{"line_number":581,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"9aa7fdbe_bec5698f","line":578,"updated":"2014-10-24 10:15:15.000000000","message":"I was told a few days ago that the preferred form here is:\n\n   if [[ -n \"$GIT_DEPTH\" ]]; then\n\nfor sake of explicitness, but I wouldn\u0027t call it a blocker.","commit_id":"51f0de5c5d5aa779db0ed647804d7d9488a9122b"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"95bd7c5ec2d1d0b4e9e2f0c1585e51a10a10b0fd","unresolved":false,"context_lines":[{"line_number":576,"context_line":"    RECLONE\u003d$(trueorfalse False $RECLONE)"},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"    if [[ \"$GIT_DEPTH\" ]]; then"},{"line_number":579,"context_line":"        git_clone_flags\u003d\"$git_clone_flags --depth $GIT_DEPTH\""},{"line_number":580,"context_line":"    fi"},{"line_number":581,"context_line":""},{"line_number":582,"context_line":"    if [[ \"$OFFLINE\" \u003d \"True\" ]]; then"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"9aa7fdbe_da77799e","line":579,"updated":"2014-10-27 00:23:58.000000000","message":"can\u0027t really be set already here...","commit_id":"51f0de5c5d5aa779db0ed647804d7d9488a9122b"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"50d3adf1eb9cbd8baf26b483d20cb106fae33eea","unresolved":false,"context_lines":[{"line_number":576,"context_line":"    RECLONE\u003d$(trueorfalse False $RECLONE)"},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"    if [[ \"$GIT_DEPTH\" ]]; then"},{"line_number":579,"context_line":"        git_clone_flags\u003d\"$git_clone_flags --depth $GIT_DEPTH\""},{"line_number":580,"context_line":"    fi"},{"line_number":581,"context_line":""},{"line_number":582,"context_line":"    if [[ \"$OFFLINE\" \u003d \"True\" ]]; then"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"9aa7fdbe_11e484df","line":579,"in_reply_to":"9aa7fdbe_da77799e","updated":"2014-10-27 09:36:28.000000000","message":"yea, i tend to write it that way as it makes no difference in the single case and makes it easier to add new ones later.","commit_id":"51f0de5c5d5aa779db0ed647804d7d9488a9122b"}]}