)]}'
{"tools/layout-checks.py":[{"author":{"_account_id":6316,"name":"Anita Kuno","email":"anteaya@anteaya.info","username":"anteaya"},"change_message_id":"3a22d9f844b500e03fc7e651a1bebbbbb8ca6cc6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/usr/bin/python"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2014 OpenStack Foundation"},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":6,"context_line":"# you may not use this file except in compliance with the License."}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_5f46c2c0","line":3,"updated":"2014-09-30 14:20:49.000000000","message":"If this is a new file shouldn\u0027t the license header include the employer of the author?","commit_id":"0954d2de239638dc88f90d4b7ab4158b94b9d87e"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"5ab9d004a2963e9c91f871aafffb1ba8f0d07a2c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/usr/bin/python"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2014 OpenStack Foundation"},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":6,"context_line":"# you may not use this file except in compliance with the License."}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_7e89b3c9","line":3,"in_reply_to":"baa201ad_5f46c2c0","updated":"2014-09-30 15:53:06.000000000","message":"The first part of it was done by Jeremy - and therefore the above should be fine. Let me add SUSE as well...","commit_id":"0954d2de239638dc88f90d4b7ab4158b94b9d87e"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"44eceb80f695c75fb327d15235eef7fb67f494a1","unresolved":false,"context_lines":[{"line_number":34,"context_line":"                if template[\u0027name\u0027] \u003d\u003d \u0027merge-check\u0027:"},{"line_number":35,"context_line":"                    correct \u003d True"},{"line_number":36,"context_line":"            if not correct:"},{"line_number":37,"context_line":"                raise"},{"line_number":38,"context_line":"        except:"},{"line_number":39,"context_line":"            print(\"Project %s has no merge-check template\" % project[\u0027name\u0027])"},{"line_number":40,"context_line":"            errors \u003d True"}],"source_content_type":"text/x-python","patch_set":5,"id":"baa201ad_cd7e73ae","line":37,"updated":"2014-09-30 18:58:52.000000000","message":"What are we raising here? Shouldn\u0027t we raise an Exception with a specific error message?","commit_id":"c6b1ffd04f07c7c81828de8ab63bc88222e68bde"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"44eceb80f695c75fb327d15235eef7fb67f494a1","unresolved":false,"context_lines":[{"line_number":37,"context_line":"                raise"},{"line_number":38,"context_line":"        except:"},{"line_number":39,"context_line":"            print(\"Project %s has no merge-check template\" % project[\u0027name\u0027])"},{"line_number":40,"context_line":"            errors \u003d True"},{"line_number":41,"context_line":"    return errors"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"baa201ad_8d5b8b53","line":40,"updated":"2014-09-30 18:58:52.000000000","message":"Or are we just going to use this exception catching block to deal with that?","commit_id":"c6b1ffd04f07c7c81828de8ab63bc88222e68bde"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"fec11d8bfec30ee98743a07b7d0518e1c509c70d","unresolved":false,"context_lines":[{"line_number":37,"context_line":"                raise"},{"line_number":38,"context_line":"        except:"},{"line_number":39,"context_line":"            print(\"Project %s has no merge-check template\" % project[\u0027name\u0027])"},{"line_number":40,"context_line":"            errors \u003d True"},{"line_number":41,"context_line":"    return errors"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"baa201ad_f8e9331b","line":40,"in_reply_to":"baa201ad_8d5b8b53","updated":"2014-09-30 19:00:41.000000000","message":"Exactly.\n\nBut we could merge it as well...","commit_id":"c6b1ffd04f07c7c81828de8ab63bc88222e68bde"}]}