)]}'
{"api-guide/source/conf.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"63d26b4ec906b360c460e1f857a12e3509f983db","unresolved":false,"context_lines":[{"line_number":85,"context_line":"   \"-d\u0027 \u0027\"]"},{"line_number":86,"context_line":"gitsha \u003d subprocess.Popen(git_cmd,"},{"line_number":87,"context_line":"                          stdout\u003dsubprocess.PIPE).\\"},{"line_number":88,"context_line":"                          communicate()[0].strip(\u0027\\n\u0027)"},{"line_number":89,"context_line":"# source tree"},{"line_number":90,"context_line":"pwd \u003d subprocess.Popen(\"pwd\", stdout\u003dsubprocess.PIPE).\\"},{"line_number":91,"context_line":"                       communicate()[0].strip(\u0027\\n\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_8074869c","line":88,"updated":"2015-12-30 17:44:12.000000000","message":"This isn\u0027t going to work.  This is going to execute \"/usr/bin/git\" with the verbatim arguments \"log | head -n1 | cut -f2 -d\u0027 \u0027\", which is an invalid git command.","commit_id":"c79f42732305069f77e2f3f12d9bb9ee8c7e1c2d"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"026b19066f8b2dafeea08e950ac91df775130578","unresolved":false,"context_lines":[{"line_number":85,"context_line":"gitsha \u003d subprocess.Popen("},{"line_number":86,"context_line":"    git_cmd, stdout\u003dsubprocess.PIPE).communicate()[0]"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"gitsha \u003d gitsha.split(\u0027\\n\u0027)[0].split(\u0027 \u0027)[1]"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"# source tree"},{"line_number":91,"context_line":"pwd \u003d subprocess.Popen(\"pwd\", stdout\u003dsubprocess.PIPE).\\"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa69d971_cbc74c62","line":88,"updated":"2016-01-06 17:03:42.000000000","message":"Much better; I\u0027m glad to be rid of that pipeline :)\n\nIncidentally, I wonder whether it\u0027d be worth replacing this with the simple command \"git rev-parse HEAD\", which does essentially the same thing as the pipeline was intended to accomplish?\n\n    [klmitch@einstein nova]$ git rev-parse HEAD\n    fd8f044d044763c8d78be654b72531f293d832af\n\nMaybe something for a separate change *shrug*","commit_id":"8242b379786b9330f4fa10cd7d8293633514be4a"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"026b19066f8b2dafeea08e950ac91df775130578","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"# source tree"},{"line_number":91,"context_line":"pwd \u003d subprocess.Popen(\"pwd\", stdout\u003dsubprocess.PIPE).\\"},{"line_number":92,"context_line":"                       communicate()[0].strip(\u0027\\n\u0027)"},{"line_number":93,"context_line":"# html_context allows us to pass arbitrary values into the html template"},{"line_number":94,"context_line":"html_context \u003d {\"pwd\": pwd,"},{"line_number":95,"context_line":"                \"gitsha\": gitsha,"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa69d971_ebbed0ac","line":92,"updated":"2016-01-06 17:03:42.000000000","message":"Personally, I don\u0027t like backslash continuations, though this is one of the uses that I tend to not object to (tons of these in the SQL code).  However, I wonder if I could get you to consider changing the indentation here, i.e.:\n\n    pwd \u003d subprocess.Popen(\"pwd\", stdout\u003dsubprocess.PIPE).\\\n        communicate()[0].strip(\u0027\\n\u0027)","commit_id":"8242b379786b9330f4fa10cd7d8293633514be4a"}],"doc/source/conf.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"026b19066f8b2dafeea08e950ac91df775130578","unresolved":false,"context_lines":[{"line_number":196,"context_line":"# using the given strftime format."},{"line_number":197,"context_line":"#html_last_updated_fmt \u003d \u0027%b %d, %Y\u0027"},{"line_number":198,"context_line":"git_cmd \u003d [\"git\", \"log\", \"--pretty\u003dformat:\u0027%ad, commit %h\u0027\", \"--date\u003dlocal\""},{"line_number":199,"context_line":"   \"-n1\"]"},{"line_number":200,"context_line":"html_last_updated_fmt \u003d subprocess.Popen(git_cmd,"},{"line_number":201,"context_line":"                                         stdout\u003dsubprocess.PIPE).\\"},{"line_number":202,"context_line":"                                         communicate()[0]"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa69d971_6b8500cd","line":199,"updated":"2016-01-06 17:03:42.000000000","message":"Strictly speaking, I\u0027d like to see the indentation line up like:\n\n    git_cmd \u003d [\"git\", \"log\", \"--pretty\u003dformat:\u0027%ad, commit %h\u0027\",\n               \"--date\u003dlocal-n1\"]\n\nRather than breaking that last argument across lines like you have it here.  (Not sure what \"local-n1\" means when given to \"--date\"; did you typo and accidentally leave out a comma?)","commit_id":"8242b379786b9330f4fa10cd7d8293633514be4a"},{"author":{"_account_id":10984,"name":"Harshada Mangesh Kakad","email":"harshadak@metsi.co.uk","username":"harshada.kakad"},"change_message_id":"70ed9a3696506525e11a3953d45dac2f96f22f6c","unresolved":false,"context_lines":[{"line_number":196,"context_line":"# using the given strftime format."},{"line_number":197,"context_line":"#html_last_updated_fmt \u003d \u0027%b %d, %Y\u0027"},{"line_number":198,"context_line":"git_cmd \u003d [\"git\", \"log\", \"--pretty\u003dformat:\u0027%ad, commit %h\u0027\", \"--date\u003dlocal\""},{"line_number":199,"context_line":"   \"-n1\"]"},{"line_number":200,"context_line":"html_last_updated_fmt \u003d subprocess.Popen(git_cmd,"},{"line_number":201,"context_line":"                                         stdout\u003dsubprocess.PIPE).\\"},{"line_number":202,"context_line":"                                         communicate()[0]"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa69d971_c18f2b73","line":199,"in_reply_to":"fa69d971_6b8500cd","updated":"2016-01-06 17:34:35.000000000","message":"@Kevin : Thanks for your comments. Yes its type and I accidentally missed it. Will update and send new patch set.","commit_id":"8242b379786b9330f4fa10cd7d8293633514be4a"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"026b19066f8b2dafeea08e950ac91df775130578","unresolved":false,"context_lines":[{"line_number":199,"context_line":"   \"-n1\"]"},{"line_number":200,"context_line":"html_last_updated_fmt \u003d subprocess.Popen(git_cmd,"},{"line_number":201,"context_line":"                                         stdout\u003dsubprocess.PIPE).\\"},{"line_number":202,"context_line":"                                         communicate()[0]"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"# If true, SmartyPants will be used to convert quotes and dashes to"},{"line_number":205,"context_line":"# typographically correct entities."}],"source_content_type":"text/x-python","patch_set":3,"id":"fa69d971_eb4810b5","line":202,"updated":"2016-01-06 17:03:42.000000000","message":"Same comment about indentation after a backslash continuation here as on the previous file…","commit_id":"8242b379786b9330f4fa10cd7d8293633514be4a"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"01f57fe8d721ef4df34ee1389071152838cb1f92","unresolved":false,"context_lines":[{"line_number":196,"context_line":"# using the given strftime format."},{"line_number":197,"context_line":"#html_last_updated_fmt \u003d \u0027%b %d, %Y\u0027"},{"line_number":198,"context_line":"git_cmd \u003d [\"git\", \"log\", \"--pretty\u003dformat:\u0027%ad, commit %h\u0027\", \"--date\u003dlocal\","},{"line_number":199,"context_line":"    \"-n1\"]"},{"line_number":200,"context_line":"html_last_updated_fmt \u003d subprocess.Popen("},{"line_number":201,"context_line":"    git_cmd, stdout\u003dsubprocess.PIPE).communicate()[0]"},{"line_number":202,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"fa69d971_7c15c899","line":199,"updated":"2016-01-06 18:21:09.000000000","message":"Would like the indentation adjusted here to line up with the \"git\" from the line above, e.g.:\n\n    git_cmd \u003d [\"git\", \"log\", \"--pretty\u003dformat:\u0027%ad, commit %h\u0027\", \"--date\u003dlocal\",\n               \"-n1\"]\n\n(femtonit)","commit_id":"c40464514d999917aa1b1f0e00f261136e046f39"}]}
