)]}'
{"jenkins/scripts/project-requirements-change.py":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"611b4fa151d78e6dc168cc1a67dd7c95d981da19","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    print err"},{"line_number":149,"context_line":"    os.chdir(reqdir)"},{"line_number":150,"context_line":"    print \"requirements git sha: %s\" % run_command("},{"line_number":151,"context_line":"        \"git rev-parse HEAD\").strip()"},{"line_number":152,"context_line":"    os_reqs \u003d RequirementsList(\u0027openstack/requirements\u0027)"},{"line_number":153,"context_line":"    if branch \u003d\u003d \u0027master\u0027 or branch.startswith(\u0027feature/\u0027):"},{"line_number":154,"context_line":"        include_dev \u003d True"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a6ced46_9508d02e","line":151,"updated":"2015-03-24 18:30:17.000000000","message":"Should this instead be:\n\n    print \"requirements git sha: %s\" % run_command(\n        \"git rev-parse HEAD\").strip()[0]","commit_id":"9323670f58a878a0db9381b0800dd45dfc1a3390"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"e31799feb8cfdabc73b100b69218b03cf87dbcf5","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    print err"},{"line_number":149,"context_line":"    os.chdir(reqdir)"},{"line_number":150,"context_line":"    print \"requirements git sha: %s\" % run_command("},{"line_number":151,"context_line":"        \"git rev-parse HEAD\").strip()"},{"line_number":152,"context_line":"    os_reqs \u003d RequirementsList(\u0027openstack/requirements\u0027)"},{"line_number":153,"context_line":"    if branch \u003d\u003d \u0027master\u0027 or branch.startswith(\u0027feature/\u0027):"},{"line_number":154,"context_line":"        include_dev \u003d True"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a6ced46_b891419b","line":151,"in_reply_to":"1a6ced46_3859d136","updated":"2015-03-24 18:35:35.000000000","message":"Heh. Obviously I meant )[0].strip() there.","commit_id":"9323670f58a878a0db9381b0800dd45dfc1a3390"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"7d24f950be981d919c5d793cf6ba993fd9fe8de5","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    print err"},{"line_number":149,"context_line":"    os.chdir(reqdir)"},{"line_number":150,"context_line":"    print \"requirements git sha: %s\" % run_command("},{"line_number":151,"context_line":"        \"git rev-parse HEAD\").strip()"},{"line_number":152,"context_line":"    os_reqs \u003d RequirementsList(\u0027openstack/requirements\u0027)"},{"line_number":153,"context_line":"    if branch \u003d\u003d \u0027master\u0027 or branch.startswith(\u0027feature/\u0027):"},{"line_number":154,"context_line":"        include_dev \u003d True"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a6ced46_3859d136","line":151,"in_reply_to":"1a6ced46_9508d02e","updated":"2015-03-24 18:32:55.000000000","message":"1. Probably out of scope for this\n\n2. strip should just remove the whitespace, not create a list. Why would we only print the first character of the string?","commit_id":"9323670f58a878a0db9381b0800dd45dfc1a3390"}]}