)]}'
{"doc/source/conf.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"7e4098c12b99bc3399af6560e7f8c066c3346e10","unresolved":false,"context_lines":[{"line_number":79,"context_line":"# Example configuration for intersphinx: refer to the Python standard library."},{"line_number":80,"context_line":"#intersphinx_mapping \u003d {\u0027http://docs.python.org/\u0027: None}"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"git_cmd \u003d [\"git\", \"tag\"]"},{"line_number":83,"context_line":"try:"},{"line_number":84,"context_line":"    raw_version_list \u003d subprocess.Popen("},{"line_number":85,"context_line":"        git_cmd, stdout\u003dsubprocess.PIPE).communicate()[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_d4c75710","line":82,"updated":"2016-05-09 12:16:57.000000000","message":"This logic is only going to apply if a block like this is pasted into a conf.py. If you add this logic to oslopshinx/__init__.py and update the _html_page_context function there, the data will be available to all users of the theme automatically without them having to change their configuration. There\u0027s an existing example where it tries to guess the URL of the git repo. Note that in the case of an error from git, you\u0027ll want to silently return an empty list instead of reporting a warning, to avoid breaking offline builds from distros.","commit_id":"8ecadedf80e35be4ccde92e787a57b966ede8915"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"53eb442d29cfab90505adacf533a8cf355079314","unresolved":false,"context_lines":[{"line_number":79,"context_line":"# Example configuration for intersphinx: refer to the Python standard library."},{"line_number":80,"context_line":"#intersphinx_mapping \u003d {\u0027http://docs.python.org/\u0027: None}"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"git_cmd \u003d [\"git\", \"tag\"]"},{"line_number":83,"context_line":"try:"},{"line_number":84,"context_line":"    raw_version_list \u003d subprocess.Popen("},{"line_number":85,"context_line":"        git_cmd, stdout\u003dsubprocess.PIPE).communicate()[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_fce18b9b","line":82,"in_reply_to":"dab17558_73194f88","updated":"2016-05-09 21:14:33.000000000","message":"Done","commit_id":"8ecadedf80e35be4ccde92e787a57b966ede8915"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"75e0ab8f41198e5a0c0e04451f46aacc5124fc88","unresolved":false,"context_lines":[{"line_number":79,"context_line":"# Example configuration for intersphinx: refer to the Python standard library."},{"line_number":80,"context_line":"#intersphinx_mapping \u003d {\u0027http://docs.python.org/\u0027: None}"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"git_cmd \u003d [\"git\", \"tag\"]"},{"line_number":83,"context_line":"try:"},{"line_number":84,"context_line":"    raw_version_list \u003d subprocess.Popen("},{"line_number":85,"context_line":"        git_cmd, stdout\u003dsubprocess.PIPE).communicate()[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_73194f88","line":82,"in_reply_to":"dab17558_b1a6170e","updated":"2016-05-09 19:33:27.000000000","message":"I\u0027m OK with making it on by default. That\u0027s the point of having a common theme. If we get complaints, we can go back and add a boolean option to enable/disable.\n\nIf someone runs sphinx with -W I think that will trigger a failure. I don\u0027t think we have that option turned on in most of our builds because of a pbr bug.\n\nFrom inside the _html_page_context function you can use app.info() and app.warn() to emit warnings using Sphinx\u0027s built-in tooling if you like, and if it becomes a problem we can change it.","commit_id":"8ecadedf80e35be4ccde92e787a57b966ede8915"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"d1d23f0aad42da4e916b6f13485885336f6741fd","unresolved":false,"context_lines":[{"line_number":79,"context_line":"# Example configuration for intersphinx: refer to the Python standard library."},{"line_number":80,"context_line":"#intersphinx_mapping \u003d {\u0027http://docs.python.org/\u0027: None}"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"git_cmd \u003d [\"git\", \"tag\"]"},{"line_number":83,"context_line":"try:"},{"line_number":84,"context_line":"    raw_version_list \u003d subprocess.Popen("},{"line_number":85,"context_line":"        git_cmd, stdout\u003dsubprocess.PIPE).communicate()[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_b1a6170e","line":82,"in_reply_to":"dab17558_d4c75710","updated":"2016-05-09 16:29:46.000000000","message":"That is a really good idea. I can do that, but I want to be very specific about one thing: are we ok with every user of oslosphinx automatically getting this update or would we prefer that each project opt-in to it? (I really don\u0027t care, and I can see advantages to making it on-by-default.)\n\nThe warning.warn() will break distro builds? In my local testing of this it didn\u0027t look any different than other sphinx warnings, but I wasn\u0027t doing distro builds. I certainly don\u0027t want to break those builds, but I\u0027d prefer not to fail silently if it can be helped.","commit_id":"8ecadedf80e35be4ccde92e787a57b966ede8915"}],"oslosphinx/theme/openstack/layout.html":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"7e4098c12b99bc3399af6560e7f8c066c3346e10","unresolved":false,"context_lines":[{"line_number":32,"context_line":"          {%- endblock %}"},{"line_number":33,"context_line":"          {%- block otherversions %}"},{"line_number":34,"context_line":"            {%- if other_versions %}"},{"line_number":35,"context_line":"              \u003ch3\u003e{{ _(\u0027Other Versions\u0027) }}\u003c/h3\u003e"},{"line_number":36,"context_line":"              \u003cul class\u003d\"this-page-menu\"\u003e"},{"line_number":37,"context_line":"              {%- for ver in other_versions[:5]: %}"},{"line_number":38,"context_line":"                  \u003cli\u003e\u003ca href\u003d\"/{{ver}}\" rel\u003d\"nofollow\"\u003e{{ ver }}\u003c/a\u003e\u003c/li\u003e"}],"source_content_type":"text/html","patch_set":2,"id":"dab17558_54dc07e4","line":35,"updated":"2016-05-09 12:16:57.000000000","message":"We may want some sort of dropdown list instead of listing all of the versions out in a UL. Many of our projects have a *lot* of versions now.","commit_id":"8ecadedf80e35be4ccde92e787a57b966ede8915"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"75e0ab8f41198e5a0c0e04451f46aacc5124fc88","unresolved":false,"context_lines":[{"line_number":32,"context_line":"          {%- endblock %}"},{"line_number":33,"context_line":"          {%- block otherversions %}"},{"line_number":34,"context_line":"            {%- if other_versions %}"},{"line_number":35,"context_line":"              \u003ch3\u003e{{ _(\u0027Other Versions\u0027) }}\u003c/h3\u003e"},{"line_number":36,"context_line":"              \u003cul class\u003d\"this-page-menu\"\u003e"},{"line_number":37,"context_line":"              {%- for ver in other_versions[:5]: %}"},{"line_number":38,"context_line":"                  \u003cli\u003e\u003ca href\u003d\"/{{ver}}\" rel\u003d\"nofollow\"\u003e{{ ver }}\u003c/a\u003e\u003c/li\u003e"}],"source_content_type":"text/html","patch_set":2,"id":"dab17558_730dce36","line":35,"in_reply_to":"dab17558_2e95a2dd","updated":"2016-05-09 19:33:27.000000000","message":"Ah, I didn\u0027t notice the [:5] there. That should address my concerns, and if someone wants more releases we can do something about the rendering at that point.","commit_id":"8ecadedf80e35be4ccde92e787a57b966ede8915"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"d1d23f0aad42da4e916b6f13485885336f6741fd","unresolved":false,"context_lines":[{"line_number":32,"context_line":"          {%- endblock %}"},{"line_number":33,"context_line":"          {%- block otherversions %}"},{"line_number":34,"context_line":"            {%- if other_versions %}"},{"line_number":35,"context_line":"              \u003ch3\u003e{{ _(\u0027Other Versions\u0027) }}\u003c/h3\u003e"},{"line_number":36,"context_line":"              \u003cul class\u003d\"this-page-menu\"\u003e"},{"line_number":37,"context_line":"              {%- for ver in other_versions[:5]: %}"},{"line_number":38,"context_line":"                  \u003cli\u003e\u003ca href\u003d\"/{{ver}}\" rel\u003d\"nofollow\"\u003e{{ ver }}\u003c/a\u003e\u003c/li\u003e"}],"source_content_type":"text/html","patch_set":2,"id":"dab17558_2e95a2dd","line":35,"in_reply_to":"dab17558_54dc07e4","updated":"2016-05-09 16:29:46.000000000","message":"Yep. But not being a UI person myself (and realizing that UI stuff is highly opinionated and a huge time sink), I went for the easiest thing that works and simply take thee 5 most recent versions (line 37).","commit_id":"8ecadedf80e35be4ccde92e787a57b966ede8915"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"53eb442d29cfab90505adacf533a8cf355079314","unresolved":false,"context_lines":[{"line_number":32,"context_line":"          {%- endblock %}"},{"line_number":33,"context_line":"          {%- block otherversions %}"},{"line_number":34,"context_line":"            {%- if other_versions %}"},{"line_number":35,"context_line":"              \u003ch3\u003e{{ _(\u0027Other Versions\u0027) }}\u003c/h3\u003e"},{"line_number":36,"context_line":"              \u003cul class\u003d\"this-page-menu\"\u003e"},{"line_number":37,"context_line":"              {%- for ver in other_versions[:5]: %}"},{"line_number":38,"context_line":"                  \u003cli\u003e\u003ca href\u003d\"/{{ver}}\" rel\u003d\"nofollow\"\u003e{{ ver }}\u003c/a\u003e\u003c/li\u003e"}],"source_content_type":"text/html","patch_set":2,"id":"dab17558_5c96d7f0","line":35,"in_reply_to":"dab17558_730dce36","updated":"2016-05-09 21:14:33.000000000","message":"Moved the limiter (last 5) this to the __init__.py so this stays fairly generic","commit_id":"8ecadedf80e35be4ccde92e787a57b966ede8915"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"650e24040bc132047321e5f87e2c22704deefc08","unresolved":false,"context_lines":[{"line_number":35,"context_line":"              \u003ch3\u003e{{ _(\u0027Other Versions\u0027) }}\u003c/h3\u003e"},{"line_number":36,"context_line":"              \u003cul class\u003d\"this-page-menu\"\u003e"},{"line_number":37,"context_line":"              {%- for ver in other_versions: %}"},{"line_number":38,"context_line":"                  \u003cli\u003e\u003ca href\u003d\"/{{ver}}\" rel\u003d\"nofollow\"\u003e{{ ver }}\u003c/a\u003e\u003c/li\u003e"},{"line_number":39,"context_line":"              {%- endfor %}"},{"line_number":40,"context_line":"              \u003c/ul\u003e"},{"line_number":41,"context_line":"            {%- endif %}"}],"source_content_type":"text/html","patch_set":4,"id":"9abb7d3a_17b95d2e","line":38,"range":{"start_line":38,"start_character":30,"end_line":38,"end_character":39},"updated":"2016-06-01 16:53:13.000000000","message":"This gives a link to the top level, e.g. docs.openstack.org/4.3.0 - see docs-draft for an example.","commit_id":"75cfb9f93b3560ff412cae4de3029157c14b258f"}]}
