Skip to content

Install development dependencies for Read the Docs dev builds - #79

Merged
PierreRaybaut merged 1 commit into
developfrom
ci/rtd-dev-deps
Oct 23, 2026
Merged

PierreRaybaut merged 1 commit into
developfrom
ci/rtd-dev-deps

Conversation

@PierreRaybaut

Copy link
Copy Markdown
Contributor

Summary

Read the Docs installs dependencies from PyPI only. Branch and pull request documentation builds therefore fail as soon as Sigima requires a guidata version that is not released yet. This happened on every SigimaX pull request of the last development cycle (fixed by DataLab-Platform/SigimaX#14).

Changes

  • New scripts/rtd_install_dev_deps.py, run as a build.jobs.pre_install step, mirroring test.yml:
    • Tag builds and main (stable / latest): unchanged, dependencies come from PyPI.
    • Other branches and pull requests: guidata is installed from its develop branch. RTD's regular install step then keeps it, as it satisfies the requirement.
  • The same script is used by SigimaX and PlotPy; only its project-specific settings block differs.

Validation

  • Branch selection checked for a tag, the default branch, develop, a pull request and a pull request from a fork.
  • Read the Docs install sequence reproduced on Ubuntu 22.04 (same image as RTD) in a clean virtualenv: the development dependencies are installed from GitHub, then RTD's pip install --upgrade --upgrade-strategy only-if-needed .[doc] keeps them and succeeds.

Read the Docs settings

Pull request builds are not enabled yet for sigima and sigima-fr: enable Build pull requests for this project in both projects to get the documentation check on pull requests.

… builds

Read the Docs installs dependencies from PyPI only, so documentation builds of
development branches and pull requests fail as soon as Sigima requires a
guidata version that is not released yet.

Add a pre_install step mirroring the GitHub Actions test workflow: for branch
and pull request builds, guidata is installed from its develop branch. Tag
builds and the main branch keep using PyPI releases.

The script is shared with PlotPy and SigimaX; only its project-specific
settings differ.

Assisted-by: Claude Opus 5.5
@PierreRaybaut PierreRaybaut reopened this Oct 23, 2026
@PierreRaybaut
PierreRaybaut merged commit ba3e4cf into develop Oct 23, 2026
8 checks passed
@PierreRaybaut
PierreRaybaut deleted the ci/rtd-dev-deps branch October 23, 2026 14:33
@ThomasMalletCodra ThomasMalletCodra added this to the v1.4.0 milestone Oct 27, 2026
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