Skip to content

[doc] Reuse cached artifacts.#11560

Merged
trivialfis merged 2 commits into
dmlc:masterfrom
trivialfis:doc-reuse-artifacts
Jul 17, 2025
Merged

[doc] Reuse cached artifacts.#11560
trivialfis merged 2 commits into
dmlc:masterfrom
trivialfis:doc-reuse-artifacts

Conversation

@trivialfis

@trivialfis trivialfis commented Jul 14, 2025

Copy link
Copy Markdown
Member

Let the conf script download the artifacts to the specified locations for reuse.

@trivialfis trivialfis requested review from Copilot and hcho3 July 14, 2025 09:03

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.

Pull Request Overview

This PR enhances the documentation build system to support reusing cached artifacts by downloading them to specified local paths when they don't exist. The main goal is to improve efficiency by avoiding redundant downloads when artifacts are already available locally.

Key changes:

  • Modified environment variable behavior to support caching artifacts at specified paths
  • Added logic to check for existing artifacts before downloading
  • Updated documentation to reflect the new caching functionality

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
doc/contrib/docs.rst Updated documentation to explain new caching behavior for R and JVM docs environment variables
doc/conf.py Implemented caching logic in both R and JVM document fetching functions

Comment thread doc/contrib/docs.rst Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@trivialfis trivialfis merged commit d603953 into dmlc:master Jul 17, 2025
62 checks passed
@trivialfis trivialfis deleted the doc-reuse-artifacts branch July 17, 2025 08:05
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