)]}'
{"tools/aclmanager.py":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"f15eb07ac3c302ef46040bce342567516c117904","unresolved":false,"context_lines":[{"line_number":146,"context_line":"    # Build the list of calls to make"},{"line_number":147,"context_line":"    print(\u0027Computing the list of modifications\u0027)"},{"line_number":148,"context_line":"    calls \u003d set()"},{"line_number":149,"context_line":"    for team, _ in repositories_list():"},{"line_number":150,"context_line":"        group \u003d \u0027%s-release-branch\u0027 % team"},{"line_number":151,"context_line":"        for (verb, memberformat) in actions:"},{"line_number":152,"context_line":"            member \u003d memberformat(team)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5201f7_38914d9a","side":"PARENT","line":149,"range":{"start_line":149,"start_character":19,"end_line":149,"end_character":38},"updated":"2017-01-11 03:49:47.000000000","message":"How\u0027d that ever work?","commit_id":"c22f58ae3b6c917e9018e4b27bbad49fc0a59972"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"af207a56bf562b33f639cdad9ed6c8e432677dcf","unresolved":false,"context_lines":[{"line_number":146,"context_line":"    # Build the list of calls to make"},{"line_number":147,"context_line":"    print(\u0027Computing the list of modifications\u0027)"},{"line_number":148,"context_line":"    calls \u003d set()"},{"line_number":149,"context_line":"    for team, _ in repositories_list():"},{"line_number":150,"context_line":"        group \u003d \u0027%s-release-branch\u0027 % team"},{"line_number":151,"context_line":"        for (verb, memberformat) in actions:"},{"line_number":152,"context_line":"            member \u003d memberformat(team)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5201f7_d2e9d2b8","side":"PARENT","line":149,"range":{"start_line":149,"start_character":19,"end_line":149,"end_character":38},"in_reply_to":"ba5201f7_38914d9a","updated":"2017-01-11 18:58:08.000000000","message":"Oh, _ is a legal identifier name. It\u0027s usually used when a value can be ignored. The important change was to add the args to the repositories_list() call, and I just copied and pasted another call here which resulted in replacing the variable name, too.","commit_id":"c22f58ae3b6c917e9018e4b27bbad49fc0a59972"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"650dbe42677e459bde3268643d42f22843807816","unresolved":false,"context_lines":[{"line_number":146,"context_line":"    # Build the list of calls to make"},{"line_number":147,"context_line":"    print(\u0027Computing the list of modifications\u0027)"},{"line_number":148,"context_line":"    calls \u003d set()"},{"line_number":149,"context_line":"    for team, _ in repositories_list():"},{"line_number":150,"context_line":"        group \u003d \u0027%s-release-branch\u0027 % team"},{"line_number":151,"context_line":"        for (verb, memberformat) in actions:"},{"line_number":152,"context_line":"            member \u003d memberformat(team)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5201f7_61613239","side":"PARENT","line":149,"range":{"start_line":149,"start_character":19,"end_line":149,"end_character":38},"in_reply_to":"ba5201f7_d2e9d2b8","updated":"2017-01-11 20:46:37.000000000","message":"I get the variable name thing I was wondering how repositories_list() without any args worked.  I\u0027d have expected pep8 to error out etc.\n\nAnyway it isn\u0027t terribly important.  This review is clearly correct.","commit_id":"c22f58ae3b6c917e9018e4b27bbad49fc0a59972"}]}