)]}'
{"doc/ext/driver_opts.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"c1a28a055f9554f71520c96edf6d264bb498a4a9","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from docutils import nodes"},{"line_number":21,"context_line":"from docutils.parsers import rst"},{"line_number":22,"context_line":"from docutils.parsers.rst import directives"},{"line_number":23,"context_line":"from docutils.statemachine import ViewList"},{"line_number":24,"context_line":"from oslo_config import cfg"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f6a8fd7_403956ef","line":23,"updated":"2018-04-25 18:42:32.000000000","message":"Should we import modules here or it\u0027s just a copy from some other project?","commit_id":"8165cb46bd505a8454305f7ff3beb1859323535f"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4d492b8dafdda56eae54bb7811c49e782e2f5b92","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from docutils import nodes"},{"line_number":21,"context_line":"from docutils.parsers import rst"},{"line_number":22,"context_line":"from docutils.parsers.rst import directives"},{"line_number":23,"context_line":"from docutils.statemachine import ViewList"},{"line_number":24,"context_line":"from oslo_config import cfg"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f6a8fd7_a32ddc95","line":23,"in_reply_to":"9f6a8fd7_403956ef","updated":"2018-04-25 18:46:42.000000000","message":"I did use some existing sphinx extension examples as a starting point. I can change this here if you\u0027d prefer. I was just following documented examples.","commit_id":"8165cb46bd505a8454305f7ff3beb1859323535f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"c1a28a055f9554f71520c96edf6d264bb498a4a9","unresolved":false,"context_lines":[{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    def run(self):"},{"line_number":70,"context_line":"        \"\"\"Load and find config options to document.\"\"\""},{"line_number":71,"context_line":"        modules \u003d ["},{"line_number":72,"context_line":"            c.strip()"},{"line_number":73,"context_line":"            for c in self.content"},{"line_number":74,"context_line":"            if c.strip()"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f6a8fd7_c02c86ad","line":71,"updated":"2018-04-25 18:42:32.000000000","message":"IMO, list comprehension looks more readable as one line statement but I won\u0027t block patch for it","commit_id":"8165cb46bd505a8454305f7ff3beb1859323535f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"32b99ec71f2c954712402d40c233a968ada17cd6","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from docutils import nodes"},{"line_number":21,"context_line":"from docutils.parsers import rst"},{"line_number":22,"context_line":"from docutils.parsers.rst import directives"},{"line_number":23,"context_line":"from docutils import statemachine as sm"},{"line_number":24,"context_line":"from oslo_config import cfg"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f6a8fd7_b4797994","line":23,"updated":"2018-04-26 19:11:00.000000000","message":"Thank you for changing it. It\u0027s better to follow hacking guidelines across all the code","commit_id":"c87da917571aa2795863f78486ba5d1d6c19fce1"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"32b99ec71f2c954712402d40c233a968ada17cd6","unresolved":false,"context_lines":[{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    def run(self):"},{"line_number":70,"context_line":"        \"\"\"Load and find config options to document.\"\"\""},{"line_number":71,"context_line":"        modules \u003d [c.strip() for c in self.content if c.strip()]"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        if not modules:"},{"line_number":74,"context_line":"            raise self.error(\u0027No modules provided to document.\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f6a8fd7_546eddc9","line":71,"updated":"2018-04-26 19:11:00.000000000","message":"Thank you for changing it","commit_id":"c87da917571aa2795863f78486ba5d1d6c19fce1"}]}