Fix building documentation for Review Board 5.

Review Request #12153 — Created March 14, 2022 and submitted

Information

Review Board
release-5.0.x

Reviewers

The Review Board 5 docs failed to build due to two problems:

  1. ID collisions in the auth_permissions fixtures for the docs.db.
    The fixtures no longer contain any permissions, allowing Django to
    manage this completely.

  2. An AttributeError in the webapidocs Sphinx extension, where
    node.slice.value was failing when node.slice was a Name. In
    prior versions of Python, we'd get an ast.Index when we hit this
    code path, but no longer. The code now special-cases ast.Index,
    and has a suitable fallback for other types.

Successfully built the docs.

Found the spots where the API docs were hitting this code path. Verified
the resulting documentation still looked correct.

Summary ID
Fix building documentation for Review Board 5.
The Review Board 5 docs failed to build due to two problems: 1. ID collisions in the `auth_permissions` fixtures for the `docs.db`. The fixtures no longer contain any permissions, allowing Django to manage this completely. 2. An `AttributeError` in the `webapidocs` Sphinx extension, where `node.slice.value` was failing when `node.slice` was a `Name`. In prior versions of Python, we'd get an `ast.Index` when we hit this code path, but no longer. The code now special-cases `ast.Index`, and has a suitable fallback for other types.
476745ff35f2329144b9d02cc9a9a1462bab5b66
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-5.0.x (da41fee)
Loading...