You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2019/03/01 00:18:14 UTC

[GitHub] ocket8888 opened a new pull request #3368: Added docs dependencies needed for auto-documenting ATC Python modules

ocket8888 opened a new pull request #3368: Added docs dependencies needed for auto-documenting ATC Python modules
URL: https://github.com/apache/trafficcontrol/pull/3368
 
 
   ## Which issue is fixed by this PR? If not related to an existing issue, what does this PR do?
   Adds the dependencies of the Python client and ORT.py to `docs/sources/requirements.txt`. This is required for proper auto-documentation of those packages. To be clear, the regular documentation regard e.g. the Traffic Ops API can still be built without installing this full dependency stack, but the output will not include documentation of the Python client and ORT.py (which is, at the time of this writing, the case with our public ReadTheDocs hosting).
   
   ## Which TC components are affected by this PR?
   
   - [x] Documentation
   
   ## What is the best way to verify this PR? Please include manual steps or automated tests. 
   There are two ways to test this, one easier than the other.
   
   ### Docker
   The easiest way to verify this is to build the docs using Docker. This can be done with simply `./pkg docs` from the repository root (may need elevated permissions). This will output a Gzipped tarball of the rendered documentation in `dist/`.
   
   ### Manually
   To build the docs manually, first navigate to the documentation directory, then install the documentation dependencies from `docs/source/requirements.txt` and finally build the documentation with `make`.
   
   E.g.
   ```bash
   # Assuming we start in the repository root
   cd docs
   
   # If `pip` is not installed, it can be installed with `sudo python3 -m ensurepip`
   sudo python3 -m pip install --upgrade -r source/requirements.txt
   
   make
   ```
   ## Check all that apply
   - [x] This PR includes documentation updates

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services