Skip to content

GEOPY-2456: Solve warning issues raised by RTD#899

Open
MatthieuCMira wants to merge 1 commit into
release/GA_4.8from
GEOPY-2456c
Open

GEOPY-2456: Solve warning issues raised by RTD#899
MatthieuCMira wants to merge 1 commit into
release/GA_4.8from
GEOPY-2456c

Conversation

@MatthieuCMira
Copy link
Copy Markdown
Contributor

@MatthieuCMira MatthieuCMira commented May 22, 2026

GEOPY-2456 - Solve warning issues raised by RTD
add toctree to remove local warnings

Copilot AI review requested due to automatic review settings May 22, 2026 18:13
@github-actions github-actions Bot changed the title GEOPY-2456 GEOPY-2456: Solve warning issues raised by RTD May 22, 2026
Copy link
Copy Markdown
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

Adds missing Sphinx toctree directives to key documentation index pages so ReadTheDocs/Sphinx includes the relevant sections and stops emitting “not in any toctree/orphan” style warnings.

Changes:

  • Added a top-level toctree to docs/index.rst to explicitly include major documentation sections.
  • Added section-level toctree blocks to the UI.JSON format docs and GEOH5 format docs (including the Integrator subsection).
  • Wired Integrator subsection pages (data, groups, objects, themes) into its index.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
docs/index.rst Adds a root toctree listing primary documentation entry points.
docs/content/uijson_format/index.rst Adds a toctree to include UI.JSON subpages.
docs/content/geoh5_format/index.rst Adds a toctree to include GEOH5 format chapters and Integrator index.
docs/content/geoh5_format/integrator/index.rst Adds a toctree to include Integrator subsection pages.

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

@MatthieuCMira MatthieuCMira requested a review from domfournier May 22, 2026 18:23
@MatthieuCMira
Copy link
Copy Markdown
Contributor Author

The warnings seems to come from the api autobuild.
Anything else?

Copy link
Copy Markdown
Contributor

@domfournier domfournier left a comment

Choose a reason for hiding this comment

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

Those changes are not solving any of the RTD warnings, and also duplicating the list of entries... revert changes

Comment on lines +25 to +31
workspace
groups
objects
data
types
giftools
integrator/index
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

THis making the TOC to duplicate the list

Image

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.

3 participants