)]}'
{"openstack_governance/yamlutils.py":[{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"29770e6e8c8322266f8427a415190baf0697ff30","unresolved":false,"context_lines":[{"line_number":20,"context_line":"import yamlordereddictloader"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"_LIKE_A_NUMBER \u003d re.compile(\u0027^[0-9]+.[0-9]+$\u0027)"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def _has_newline(data):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_0216a4f4","line":23,"updated":"2018-11-02 16:35:06.000000000","message":"This regexp matches strings like \"3A2\" (which is not a number) but not \"82\" (which is a number), so I\u0027m not sure it fits its purpose.","commit_id":"566de6f2ce5a05321428c9223ba55f45bedbc143"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"4800a206f4f6934701fa1b2c6efff1f4ba7c661f","unresolved":false,"context_lines":[{"line_number":20,"context_line":"import yamlordereddictloader"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"_LIKE_A_NUMBER \u003d re.compile(\u0027^[0-9]+.[0-9]+$\u0027)"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def _has_newline(data):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_5877a88f","line":23,"in_reply_to":"3f79a3b5_0216a4f4","updated":"2018-11-02 18:32:18.000000000","message":"Interesting. I got this code from the releases repo, so we\u0027ll want to fix it there, too. I suppose that\u0027s what I get for taking untested code. ;-)\n\nI wonder if that . is meant to be escaped? Or optional?","commit_id":"566de6f2ce5a05321428c9223ba55f45bedbc143"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"9f39ff2f87f321e078957733915e0b0b29818129","unresolved":false,"context_lines":[{"line_number":20,"context_line":"import yamlordereddictloader"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"_LIKE_A_NUMBER \u003d re.compile(\u0027^[0-9]+.[0-9]+$\u0027)"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def _has_newline(data):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_e5bc7d08","line":23,"in_reply_to":"3f79a3b5_5877a88f","updated":"2018-11-07 13:33:18.000000000","message":"None of the dumping code is used here right now anyway, so I just removed it.","commit_id":"566de6f2ce5a05321428c9223ba55f45bedbc143"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"4800a206f4f6934701fa1b2c6efff1f4ba7c661f","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        if _has_newline(data):"},{"line_number":71,"context_line":"            style \u003d \"|\""},{"line_number":72,"context_line":"        elif _LIKE_A_NUMBER.match(data):"},{"line_number":73,"context_line":"            style \u003d \u0027\"\u0027"},{"line_number":74,"context_line":"        return yaml.representer.ScalarNode(\u0027tag:yaml.org,2002:str\u0027,"},{"line_number":75,"context_line":"                                           data, style\u003dstyle)"},{"line_number":76,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_78ef041c","line":73,"updated":"2018-11-02 18:32:18.000000000","message":"I don\u0027t know why we would set the style to use quotes if it is a number. It seems like we\u0027d want to do that if it is *not* a number. So maybe this is wrong, too.","commit_id":"566de6f2ce5a05321428c9223ba55f45bedbc143"}]}