Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Commit API docs and generate them using a separate task. #48

Merged
merged 10 commits into from
Mar 15, 2024
17 changes: 16 additions & 1 deletion docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,14 +21,29 @@ documentation website for this package:
```

* The output of the build will be in `../build/docs/html`.
* API reference docs will be written to `src/api`.

* Clean up the build:

```shell
task docs:clean
```

* Generate API docs:

```shell
task docs:api-docs
```

* This will regenerate the API docs in `src/api`. You should run this whenever you change the
modules or package that exist in `clp-ffi-py`, and then commit the changes.
* This will overwrite any existing API doc files. This is only a problem if any of the previously
generated files were manually customized (e.g., to reformat some docs). You should
review the changes carefully to make sure any customizations have not been mistakenly deleted.
* This will *not* remove docs for packages/modules that no longer exist. You should make sure to
delete those docs from `src/api`. In addition, ensure that:
* in the case of a rename, any previous customizations are retained.
* no unrelated files (e.g., indexes) are deleted.

## Viewing the Output

```shell
Expand Down
7 changes: 7 additions & 0 deletions docs/src/api/clp_ffi_py.ir.native.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
clp\_ffi\_py.ir.native module
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no idea what are those so I assume they are auto generated

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yessir.

=============================

.. automodule:: clp_ffi_py.ir.native
:members:
:undoc-members:
:show-inheritance:
7 changes: 7 additions & 0 deletions docs/src/api/clp_ffi_py.ir.query_builder.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
clp\_ffi\_py.ir.query\_builder module
=====================================

.. automodule:: clp_ffi_py.ir.query_builder
:members:
:undoc-members:
:show-inheritance:
7 changes: 7 additions & 0 deletions docs/src/api/clp_ffi_py.ir.readers.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
clp\_ffi\_py.ir.readers module
==============================

.. automodule:: clp_ffi_py.ir.readers
:members:
:undoc-members:
:show-inheritance:
20 changes: 20 additions & 0 deletions docs/src/api/clp_ffi_py.ir.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
clp\_ffi\_py.ir package
=======================

Submodules
----------

.. toctree::
:maxdepth: 4

clp_ffi_py.ir.native
clp_ffi_py.ir.query_builder
clp_ffi_py.ir.readers

Module contents
---------------

.. automodule:: clp_ffi_py.ir
:members:
:undoc-members:
:show-inheritance:
27 changes: 27 additions & 0 deletions docs/src/api/clp_ffi_py.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
clp\_ffi\_py package
====================

Subpackages
-----------

.. toctree::
:maxdepth: 4

clp_ffi_py.ir

Submodules
----------

.. toctree::
:maxdepth: 4

clp_ffi_py.utils
clp_ffi_py.wildcard_query

Module contents
---------------

.. automodule:: clp_ffi_py
:members:
:undoc-members:
:show-inheritance:
7 changes: 7 additions & 0 deletions docs/src/api/clp_ffi_py.utils.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
clp\_ffi\_py.utils module
=========================

.. automodule:: clp_ffi_py.utils
:members:
:undoc-members:
:show-inheritance:
7 changes: 7 additions & 0 deletions docs/src/api/clp_ffi_py.wildcard_query.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
clp\_ffi\_py.wildcard\_query module
===================================

.. automodule:: clp_ffi_py.wildcard_query
:members:
:undoc-members:
:show-inheritance:
21 changes: 13 additions & 8 deletions docs/tasks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,19 @@ vars:
sh: python3 -c "import sys; print(f'{sys.version_info.major}.{sys.version_info.minor}')"

tasks:
api-docs:
dir: "{{.TASKFILE_DIR}}"
deps: [ "docs-venv" ]
cmds:
- |-
. "{{.DOCS_VENV_DIR}}/bin/activate"
sphinx-apidoc \
--separate \
--no-toc \
--force \
-o src/api \
"{{.DOCS_VENV_DIR}}/lib/python{{.PYTHON_VERSION}}/site-packages/clp_ffi_py/"

clean:
cmds:
- "rm -rf '{{.DOCS_BUILD_DIR}}'"
Expand All @@ -31,14 +44,6 @@ tasks:
cmds:
- |-
. "{{.DOCS_VENV_DIR}}/bin/activate"

sphinx-apidoc \
--force \
--separate \
--no-toc \
-o src/api \
"{{.DOCS_VENV_DIR}}/lib/python{{.PYTHON_VERSION}}/site-packages/clp_ffi_py/"

sphinx-build -a -c conf -b html src "{{.OUTPUT_DIR}}"
# Checksum the generated files (this command must be last)
- |-
Expand Down