Skip to content

GEOPY-3095: Add option to build downloadable zipped html docs in RTD config - #489

Merged
domfournier merged 2 commits into
developfrom
GEOPY-3095
Oct 13, 2026
Merged

domfournier merged 2 commits into
developfrom
GEOPY-3095

Conversation

@benk-mira

@benk-mira benk-mira commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

GEOPY-3095 - Add option to build downloadable zipped html docs in RTD config

Copilot AI balanced review requested due to automatic review settings October 7, 2026 17:17
@benk-mira benk-mira changed the title update radthedocs.yml to add create zipped html for builds GEOPY-3095 Oct 7, 2026
@github-actions github-actions Bot changed the title GEOPY-3095 GEOPY-3095: update radthedocs.yml to add create zipped html for builds Oct 7, 2026
@github-actions github-actions Bot changed the title GEOPY-3095: update radthedocs.yml to add create zipped html for builds GEOPY-3095: Add option to build downloadable zipped html docs in RTD config Oct 7, 2026

Copilot AI left a comment

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.

🟡 Changes recommended

The formats setting must be moved to the configuration’s top level.

1 open finding
What changed in this PR

Adds downloadable zipped HTML documentation to Read the Docs.

Changes:

  • Requests the htmlzip documentation format.
File Description
.readthedocs.yml Configures zipped HTML output.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .readthedocs.yml Outdated
Comment on lines +33 to +34
formats:
- htmlzip
@domfournier
domfournier merged commit f7ad372 into develop Oct 13, 2026
21 checks passed
@domfournier
domfournier deleted the GEOPY-3095 branch October 13, 2026 19:57
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