docs: update copyright, use dynamic version, add explicit Sphinx deps#122
docs: update copyright, use dynamic version, add explicit Sphinx deps#122SmithSamuelM merged 2 commits intoioflo:mainfrom
Conversation
- Update copyright year to 2020-2026 - Switch from hardcoded version (0.3.4) to dynamic import from hio.__version__ - Add explicit Sphinx >= 8.1.3, sphinx-rtd-theme >= 3.0.1, ordered-set >= 4.1.0 to docs requirements - Fixes missing dependency warnings during doc builds - sphinx.ext.viewcode already enabled in upstream (no changes needed) Tested: cd docs && make clean && make html succeeds with 1485 warnings (RST formatting, expected)
- Remove unused sphinx_rtd_theme import - Move hio import to top of file (after sys.path setup) - Fixes Ruff F401 (unused import) and E402 (module import not at top)
|
Is napoleon. (Aka google RST format) doc strings enabled for his? Wondering if that is where the RST errors are coming from. |
|
Re: Napoleon and RST Warnings in hio Documentation Napoleon is currently enabled in hio's Sphinx configuration: extensions = [
'sphinx.ext.napoleon', # Active
# ...
]
napoleon_include_init_with_doc = TrueBuild Statistics:
Error Breakdown:
Root Cause: The docstrings appear to use indentation-based formatting that doesn't align with strict RST syntax requirements. Napoleon parses Google/NumPy style docstrings, but the underlying content still needs proper RST structure for lists, code blocks, and definition lists. Example from These occur where docstrings have code examples or parameter lists without proper RST markup (e.g., Observations: The additional Napoleon settings currently commented out could potentially improve parsing: # napoleon_use_ivar = True
# napoleon_use_rtype = False
# napoleon_use_param = FalseHowever, the core issue is structural - the docstrings themselves need RST-compliant formatting. This would require source code updates across multiple modules, which falls outside the scope of configuration changes. Recommendation: If improving documentation quality is a priority, addressing these errors would require a systematic docstring refactoring effort. The current PR focuses on configuration fixes (dynamic versioning, dependency pinning) and leaves docstring formatting as a separate initiative. Would you like me to create a tracking issue for the docstring refactoring work, or would you prefer to address this differently? |
|
Yes please make a separate tracking issue. No doubt since sphynx has not be working for some time may RST formatting has not been correct. I will need to see the nature of the errors so I can figure out what it’s complaining about since I believe I am largely following the Napolean/Google syntax. Does it still render the doc strings dispite the errors? If it does then we can accept the pull request and then have a background task of cleaning up the RST over time. |
PR: hio — Sphinx Documentation Configuration Updates (2026-01-22)
Summary of Changes
This PR updates the Sphinx documentation configuration to use dynamic versioning, adds explicit dependency pins for reproducible builds, and fixes linting issues.
Configuration Updates (
docs/source/conf.py):2020-2021to2020-20260.3.4to dynamichio.__version__sphinx_rtd_themeimport (Ruff F401)hioimport to top of file (Ruff E402)Documentation Dependencies (
docs/source/requirements.txt):Sphinx >= 8.1.3sphinx-rtd-theme >= 3.0.1ordered-set >= 4.1.0No Changes Needed:
sphinx.ext.viewcodealready enabled in upstreamioflo/hio.readthedocs.yamlalready correctly configured for Python 3.13Testing Performed
build/html/index.htmlgenerated successfullyKnown Out-of-Scope Items
The following items were noted during the audit but are not addressed in this PR:
autoapiandautodoc. These require changes to source code docstrings across multiple modules.