Skip to content

improve documentation build process by removing warnings#380

Merged
sophiamaedler merged 7 commits intomainfrom
improve_doc_warnings
Mar 1, 2026
Merged

improve documentation build process by removing warnings#380
sophiamaedler merged 7 commits intomainfrom
improve_doc_warnings

Conversation

@sophiamaedler
Copy link
Collaborator

Summary

This PR reduces Sphinx warning noise and makes docs builds more reliable.

  • Docs build tooling

    • docs/Makefile: switched to interpreter-based Sphinx invocation and improved make target handling.
    • docs/conf.py: suppressed known non-actionable config.cache warning.
  • Docs warning cleanup

    • Fixed duplicate explicit target names and broken refs in:
      • docs/index.rst
      • docs/pages/workflow.rst
      • docs/pages/workflow/segmentation_workflow.rst
    • Added :no-index: to specific autodoc blocks causing duplicate object description warnings:
      • docs/pages/module/processing.rst
      • docs/pages/module/tools/ml.rst
    • Re-enabled notebook page in toctree to avoid orphan warnings (docs/index.rst).
  • Dependency adjustments

    • Added ipython to docs requirements for notebook lexer support.
    • Added setuptools<81 to maintain current pkg_resources compatibility in dependency stack.
  • Docstring formatting

    • Cleaned numpy_to_h5sc docstring formatting in src/scportrait/io/h5sc.py to avoid docutils option-list parsing warnings.
  • Pre-commit mypy

    • Set mypy hook to --python-version=3.10 in .pre-commit-config.yaml to match project syntax/features.

Copilot AI review requested due to automatic review settings March 1, 2026 18:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to reduce Sphinx warning noise and make documentation builds more reliable by adjusting the docs build tooling, suppressing a known warning, cleaning up broken/duplicate references, and tweaking dependency/tooling configs.

Changes:

  • Updated Sphinx build invocation and configuration to reduce warning noise (docs/Makefile, docs/conf.py).
  • Cleaned up documentation references / toctrees and suppressed duplicate autodoc indexing warnings (docs/**/*.rst).
  • Adjusted dependencies and tooling configuration for docs and type checking (requirements/*.txt, .pre-commit-config.yaml).

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docs/Makefile Switches Sphinx invocation to interpreter-based module execution and improves target handling.
docs/conf.py Suppresses the config.cache warning to reduce non-actionable noise.
docs/index.rst Fixes/adjusts links and re-adds notebook page to toctree to avoid warnings.
docs/pages/workflow.rst Improves link text and references within the workflow documentation.
docs/pages/workflow/segmentation_workflow.rst Fixes a broken :ref: target for the DAPI cellpose section.
docs/pages/module/processing.rst Adds :no-index: to reduce duplicate object description warnings from autodoc.
docs/pages/module/tools/ml.rst Adds :no-index: to reduce duplicate object description warnings from autodoc.
src/scportrait/io/h5sc.py Re-formats docstring sections to avoid docutils parsing warnings.
requirements/requirements_docs.txt Adds ipython for notebook lexer support in docs builds.
requirements/requirements.txt Adds a setuptools upper bound intended to preserve pkg_resources compatibility.
.pre-commit-config.yaml Pins mypy’s target --python-version to 3.10 to match project support.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sophiamaedler sophiamaedler merged commit 7e816cf into main Mar 1, 2026
2 checks passed
@sophiamaedler sophiamaedler deleted the improve_doc_warnings branch March 1, 2026 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants