openstack/openstack-doc-tools
Code Issues Proposed changes

diff_branches: fix deprecated opts listing

Browse Source 
* Use the proper list (deprecated_opts, not new_opts)
* Sort the elements

Change-Id: I2f16ee72ed389b801590db70708db1bd4d59f88d
This commit is contained in:
Gauvain Pocentek 2015-11-18 10:14:34 +01:00
parent 8391493619
commit 19fba83f36
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
autogenerate_config_docs/diff_branches.py
View File

@ -237,7 +237,8 @@ def get_env(project, new_branch, old_list, new_list):
# Deprecated options # Deprecated options
if deprecated_opts: if deprecated_opts:
for deprecated, new in deprecated_opts: for deprecated, new in sorted(deprecated_opts, cmp=_cmpopts,
key=lambda tup: tup[0]):
deprecated = format_option_name(deprecated) deprecated = format_option_name(deprecated)
new = format_option_name(new) new = format_option_name(new)
env['deprecated_opts'].append((deprecated, new)) env['deprecated_opts'].append((deprecated, new))

2
autogenerate_config_docs/templates/changes.rst.j2
View File

@ -41,7 +41,7 @@ New, updated, and deprecated options in {{ release }} for {{ codename }}
* - Deprecated option * - Deprecated option
- New Option - New Option
{% for cells in new_opts %} {% for cells in deprecated_opts %}
* - ``{{ cells[0] }}`` * - ``{{ cells[0] }}``
- ``{{ cells[1] }}`` - ``{{ cells[1] }}``
{% endfor %} {% endfor %}