)]}'
{"doc/source/usage.rst":[{"author":{"_account_id":13734,"name":"Nell Jerram","email":"nell@tigera.io","username":"neiljerram"},"change_message_id":"a8f1b1fd40907b6f28df88bf5e9204968f2c16bb","unresolved":false,"context_lines":[{"line_number":20,"context_line":"``oslosphinx`` defines a configuration option ``oslosphinx_cgit_link``"},{"line_number":21,"context_line":"which should be the URL to the git repository browser for the project"},{"line_number":22,"context_line":"being documented. The default is a guess, and will be right for a lot"},{"line_number":23,"context_line":"of projects, but to ensure that it is correct in all situations it is"},{"line_number":24,"context_line":"best to set the value in conf.py::"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"  oslosphinx_cgit_link \u003d \u0027http://git.openstack.org/cgit/openstack/oslosphinx\u0027"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a29b11f_4783fccc","line":23,"updated":"2015-10-19 11:49:57.000000000","message":"Is \"will be right for a lot of projects\" true?  The guess is currently wrong for all the projects that I\u0027ve looked at, and, as far as I understand,\n\n- that will extend to all projects, except for any that do special things to correct this (if that\u0027s even possible, before the change that you\u0027ve proposed here)\n\n- your change here doesn\u0027t change the logic of the default guess.\n\nI very much appreciate the time you\u0027re spending on this, but I\u0027m afraid that, if this statement is incorrect as I think, it\u0027s likely to confuse people further in future.","commit_id":"c383e0026aeb9fb541486f41cf286256e312e5e3"}],"oslosphinx/__init__.py":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"22e9df180e9e6619694fac6046a0d65b9fd312eb","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        return None"},{"line_number":30,"context_line":"    else:"},{"line_number":31,"context_line":"        parsed \u003d parse.urlparse(git_remote)"},{"line_number":32,"context_line":"        return CGIT_BASE + parsed.path.lstrip(\u0027/\u0027)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"def _html_page_context(app, pagename, templatename, context, doctree):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a2ca52d_8237204c","line":32,"updated":"2015-10-16 21:26:11.000000000","message":"There\u0027s still the problem that this is mis-guessing the current URLs and is fragile to future potential changes in the mechanisms we use for cloning and updating git remotes in our CI. Would you consider a more complex alternative like:\n\n        return CGIT_BASE + re.search(\u0027/([^/]+/[^/]+)(?\u003c!\\.git)(.git|)$\u0027,\n            parsed).group(1) + \u0027/\u0027","commit_id":"c383e0026aeb9fb541486f41cf286256e312e5e3"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"6f8ebcd09ee7bc93673a7ec59c38648d0d399af7","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        return None"},{"line_number":30,"context_line":"    else:"},{"line_number":31,"context_line":"        parsed \u003d parse.urlparse(git_remote)"},{"line_number":32,"context_line":"        return CGIT_BASE + parsed.path.lstrip(\u0027/\u0027)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"def _html_page_context(app, pagename, templatename, context, doctree):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a2ca52d_0cb4a232","line":32,"in_reply_to":"5a2ca52d_8237204c","updated":"2015-10-16 22:32:58.000000000","message":"It\u0027s a guess, so I think that\u0027s ok. If this code guesses wrong, it can be overridden by setting the config option.","commit_id":"c383e0026aeb9fb541486f41cf286256e312e5e3"}]}