)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"aa1ef4238111043246a6897dc8ff322bf880c5a4","unresolved":false,"context_lines":[{"line_number":23,"context_line":"* https://review.openstack.org/#/c/70948/ : patch written for Trollus 0.1, but"},{"line_number":24,"context_line":"  it works on Trollius 0.2 too. I will update the patch to require Trollius 0.2"},{"line_number":25,"context_line":"  and use \"yield From(...)\""},{"line_number":26,"context_line":"* https://review.openstack.org/#/c/77925/ : tests are failing because this"},{"line_number":27,"context_line":"  patch requires Trollius 0.2, that\u0027s why I would like to upgrade Trollius"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"I also moved trollius to the right section of dependencies. Sorry, I didn\u0027t"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F%2F8hw%3D","line":26,"updated":"2014-03-12 19:06:59.000000000","message":"I would think this shouldn\u0027t land in Icehouse at this point in the release.","commit_id":"cda6f5b567411037b19ea1ecb1de1d1685bf2d52"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"b2c02f3257d0bcf3d01a017e6043340bbd695773","unresolved":false,"context_lines":[{"line_number":23,"context_line":"* https://review.openstack.org/#/c/70948/ : patch written for Trollus 0.1, but"},{"line_number":24,"context_line":"  it works on Trollius 0.2 too. I will update the patch to require Trollius 0.2"},{"line_number":25,"context_line":"  and use \"yield From(...)\""},{"line_number":26,"context_line":"* https://review.openstack.org/#/c/77925/ : tests are failing because this"},{"line_number":27,"context_line":"  patch requires Trollius 0.2, that\u0027s why I would like to upgrade Trollius"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"I also moved trollius to the right section of dependencies. Sorry, I didn\u0027t"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F%2F3Y8%3D","line":26,"in_reply_to":"AAAAWH%2F%2F8hw%3D","updated":"2014-03-13 09:42:33.000000000","message":"I also guess that such large change is too late for Icehouse, but I would like to allow Trollius 0.2 to be able to prepare my patches for the next release.","commit_id":"cda6f5b567411037b19ea1ecb1de1d1685bf2d52"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"aa1ef4238111043246a6897dc8ff322bf880c5a4","unresolved":false,"context_lines":[{"line_number":29,"context_line":"I also moved trollius to the right section of dependencies. Sorry, I didn\u0027t"},{"line_number":30,"context_line":"noticed first that dependencies were grouped."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"I also added newlines to make the file more readable, newlines are ignored."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Trollius has now a much better documentation:"},{"line_number":35,"context_line":"http://trollius.readthedocs.org/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F%2F8hk%3D","line":32,"updated":"2014-03-12 19:06:59.000000000","message":"I don\u0027t think we want this, at least not in this patch, it\u0027s unrelated.","commit_id":"cda6f5b567411037b19ea1ecb1de1d1685bf2d52"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"b2c02f3257d0bcf3d01a017e6043340bbd695773","unresolved":false,"context_lines":[{"line_number":29,"context_line":"I also moved trollius to the right section of dependencies. Sorry, I didn\u0027t"},{"line_number":30,"context_line":"noticed first that dependencies were grouped."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"I also added newlines to make the file more readable, newlines are ignored."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Trollius has now a much better documentation:"},{"line_number":35,"context_line":"http://trollius.readthedocs.org/"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F%2F3Yo%3D","line":32,"in_reply_to":"AAAAWH%2F%2F8hk%3D","updated":"2014-03-13 09:42:33.000000000","message":"I expected this reaction :-) And I don\u0027t want to write a separated patch to add newlines, so I just removed newlines in the new patch.","commit_id":"cda6f5b567411037b19ea1ecb1de1d1685bf2d52"}],"global-requirements.txt":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"aa1ef4238111043246a6897dc8ff322bf880c5a4","unresolved":false,"context_lines":[{"line_number":74,"context_line":"pysnmp\u003e\u003d4.2.1,\u003c5.0.0"},{"line_number":75,"context_line":"pystache"},{"line_number":76,"context_line":"python-seamicroclient\u003e\u003d0.1.0,\u003c2.0"},{"line_number":77,"context_line":"trollius\u003e\u003d0.1.4"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"# OpenStack clients. None of these should have an upper bound"},{"line_number":80,"context_line":"# as that has implications for testing in the gate. An exception"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAWH%2F%2F8jI%3D","line":77,"updated":"2014-03-12 19:06:59.000000000","message":"Let\u0027s keep the alphabetical ordering please.","commit_id":"cda6f5b567411037b19ea1ecb1de1d1685bf2d52"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"b2c02f3257d0bcf3d01a017e6043340bbd695773","unresolved":false,"context_lines":[{"line_number":74,"context_line":"pysnmp\u003e\u003d4.2.1,\u003c5.0.0"},{"line_number":75,"context_line":"pystache"},{"line_number":76,"context_line":"python-seamicroclient\u003e\u003d0.1.0,\u003c2.0"},{"line_number":77,"context_line":"trollius\u003e\u003d0.1.4"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"# OpenStack clients. None of these should have an upper bound"},{"line_number":80,"context_line":"# as that has implications for testing in the gate. An exception"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAWH%2F%2F3YE%3D","line":77,"in_reply_to":"AAAAWH%2F%2F8jI%3D","updated":"2014-03-13 09:42:33.000000000","message":"I don\u0027t understand. Why is the right place for the \"trollius\" dependency? T letter doesn\u0027t come after P?\n\nThe dependency is global, it\u0027s not specific to tests nor SQLAchemy, nor clients. Sorry, I didn\u0027t noticed the sections when I added trollius in my previous patch.","commit_id":"cda6f5b567411037b19ea1ecb1de1d1685bf2d52"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"292387044f607d7460e6d8292bc396adfe24853f","unresolved":false,"context_lines":[{"line_number":74,"context_line":"pysnmp\u003e\u003d4.2.1,\u003c5.0.0"},{"line_number":75,"context_line":"pystache"},{"line_number":76,"context_line":"python-seamicroclient\u003e\u003d0.1.0,\u003c2.0"},{"line_number":77,"context_line":"trollius\u003e\u003d0.1.4"},{"line_number":78,"context_line":"# OpenStack clients. None of these should have an upper bound"},{"line_number":79,"context_line":"# as that has implications for testing in the gate. An exception"},{"line_number":80,"context_line":"# is currently being made for neutron client because of the need"}],"source_content_type":"text/plain","patch_set":2,"id":"AAAAWH%2F%2F0ro%3D","line":77,"updated":"2014-03-13 15:29:24.000000000","message":"t comes after p, but this should come after tripleo-image-elements below :)","commit_id":"00457ea5a41ab9365d51dfeec794da680f6cc51f"}]}