Skip to content

Commit

Permalink
Restructures documentation reference section
Browse files Browse the repository at this point in the history
This section was getting unwieldly. It is now
less overwhelming.

This change does a few things:
1. breaks things out into explicit files
2. changes the sphinx build to be htmlDir to get rid of pesky .html in the URL
3. removes some to be organized docs
4. fixes up a few typos
5. fixes links to match new structure
  • Loading branch information
skrawcz committed Jul 5, 2023
1 parent e55cb4d commit 02bea1c
Show file tree
Hide file tree
Showing 72 changed files with 853 additions and 1,249 deletions.
1 change: 1 addition & 0 deletions .readthedocs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ build:

# Build documentation in the docs/ directory with Sphinx
sphinx:
builder: dirhtml
fail_on_warning: true
configuration: docs/conf.py

Expand Down
4 changes: 2 additions & 2 deletions README-DOCS.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,12 @@ pip install -r requirements-docs.txt
```
and then one of the following to build and view the documents:
```bash
sphinx-build -a docs /tmp/mydocs
sphinx-build -b dirhtml -a docs /tmp/mydocs
python -m http.server --directory /tmp/mydocs
```
or for auto rebuilding do:
```bash
sphinx-autobuild -W -E -T --watch hamilton/ -a docs /tmp/mydocs
sphinx-autobuild -b dirhtml -W -E -T --watch hamilton/ -a docs /tmp/mydocs
```
Then it'll be running on port 8000.

Expand Down
48 changes: 0 additions & 48 deletions docs/best-practices/code-organization.rst

This file was deleted.

25 changes: 0 additions & 25 deletions docs/best-practices/common-indices.rst

This file was deleted.

21 changes: 0 additions & 21 deletions docs/best-practices/function-modifiers.rst

This file was deleted.

53 changes: 0 additions & 53 deletions docs/best-practices/function-naming.rst

This file was deleted.

16 changes: 0 additions & 16 deletions docs/best-practices/index.rst

This file was deleted.

Loading

0 comments on commit 02bea1c

Please sign in to comment.