)]}'
{"keystone/common/serializer.py":[{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"85263cc7e197b6605b8b0b98fde5ec8ba7536734","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"import six"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"try:"},{"line_number":28,"context_line":"    from lxml import etree"},{"line_number":29,"context_line":"except ImportError:"},{"line_number":30,"context_line":"    etree \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_1934dc47","line":27,"updated":"2014-08-08 15:09:24.000000000","message":"are you tryingto expose try/import/catch logic first no matter whether it\u0027s a stdlib of 3rd party? Otherwise why not leave \u0027re\u0027 at the top as a stdlib.","commit_id":"7d1eed2bd5960a99897829290cdacb15c645ee28"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c9f5c146aa220862cd2f9cf38fb8af79e90204c6","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"import six"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"try:"},{"line_number":28,"context_line":"    from lxml import etree"},{"line_number":29,"context_line":"except ImportError:"},{"line_number":30,"context_line":"    etree \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_cabd1305","line":27,"in_reply_to":"baada198_1934dc47","updated":"2014-08-09 05:42:37.000000000","message":"looks fine to me, stdlib, then 3rd part (followed by a try/catch), then local imports","commit_id":"7d1eed2bd5960a99897829290cdacb15c645ee28"}]}