Add -j flag to git cl comments to get JSON summary of CL comments

git_cl.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld."""
 
 from distutils.version import LooseVersion
@@ skipping 1758 matching lines @@
     print 'Issue number: %s (%s)' % (cl.GetIssue(), cl.GetIssueURL())
   return 0
 
 
 def CMDcomments(parser, args):
   """Shows or posts review comments for any changelist."""
   parser.add_option('-a', '--add-comment', dest='comment',
                     help='comment to add to an issue')
   parser.add_option('-i', dest='issue',
                     help="review issue id (defaults to current issue)")
+  parser.add_option('-j', '--json-file',
+                    help='File to write JSON summary to')
   auth.add_auth_options(parser)
   options, args = parser.parse_args(args)
   auth_config = auth.extract_auth_config_from_options(options)
 
   issue = None
   if options.issue:
     try:
       issue = int(options.issue)
     except ValueError:
       DieWithError('A review issue id is expected to be a number')
 
   cl = Changelist(issue=issue, auth_config=auth_config)
 
   if options.comment:
     cl.AddComment(options.comment)
     return 0
 
   data = cl.GetIssueProperties()
+  summary = []
   for message in sorted(data.get('messages', []), key=lambda x: x['date']):
+    summary.append({
+        'date': message['date'],
+        'lgtm': False,
+        'message': message['text'],
+        'not lgtm': False,

[Vadim Sh., 2015/09/15 03:15:34]

nit: not_lgtm

[smut, 2015/09/15 16:18:41]

Done.

+        'sender': message['sender'],
+    })
     if message['disapproval']:
       color = Fore.RED
+      summary[-1]['not lgtm'] = True
     elif message['approval']:
       color = Fore.GREEN
+      summary[-1]['lgtm'] = True
     elif message['sender'] == data['owner_email']:
       color = Fore.MAGENTA
     else:
       color = Fore.BLUE
     print '\n%s%s  %s%s' % (
         color, message['date'].split('.', 1)[0], message['sender'],
         Fore.RESET)
     if message['text'].strip():
       print '\n'.join('  ' + l for l in message['text'].splitlines())
+  if options.json_file:
+    with open(options.json_file, 'wb') as f:
+      json.dump(summary, f)
   return 0
 
 
 def CMDdescription(parser, args):
   """Brings up the editor for the current CL's description."""
   parser.add_option('-d', '--display', action='store_true',
                     help='Display the description instead of opening an editor')
   auth.add_auth_options(parser)
   options, _ = parser.parse_args(args)
   auth_config = auth.extract_auth_config_from_options(options)
@@ skipping 1737 matching lines @@
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   colorama.init()
   try:
     sys.exit(main(sys.argv[1:]))
   except KeyboardInterrupt:
     sys.stderr.write('interrupted\n')
     sys.exit(1)