-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ci] [docs] readthedocs configuration options are deprecated #4954
Comments
Linking some relevant discussions showing support for |
Hi @jameslamb is this available to work on? |
Thanks for your interest @akshitadixit ! But no, #4957 (linked above) is already addressing this. |
* [ci] [docs] use mamba for readthedocs builds (fixes #4954) * update docs * simplify build script and add docs flag to gitignore * exit with non-0 if build fails * update CI job * add doxygen * remove outdated requirement_base.txt reference * use conda create instead of conda env create * fix conda create flags * add nodefaults to env.yml * Update docs/README.rst Co-authored-by: Nikita Titov <[email protected]> * try to fix check-docs CI job * additional changes * switch from mamba to miniforge * simplify docker command and fix issues in local build script * Apply suggestions from code review Co-authored-by: Nikita Titov <[email protected]> * update docs and conda * Apply suggestions from code review Co-authored-by: Nikita Titov <[email protected]> Co-authored-by: Nikita Titov <[email protected]>
This issue has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this. |
Description
This project uses readthedocs to build and host its documentation at https://lightgbm.readthedocs.io/en/latest/.
The way those docs are built is configured here:
LightGBM/.readthedocs.yaml
Lines 1 to 14 in 4aaeb22
According to the documentation for that service, some of the configuration options used in this project's
.readthedocs.yaml
are deprecated.python.version (link)
According to https://docs.readthedocs.io/en/stable/config-file/v2.html#python-version-legacy, using
version: 3
is also deprecated.Additional Comments
These deprecated configuration should be removed, so that documentation builds don't break when those features are removed in future changes to the readthedocs service.
Given the discussion in #4948, I think this should be accomplished by switching the project's readthedocs builds to use
mamba
, as documented at "Making builds faster with mamba" (RTD docs link).The text was updated successfully, but these errors were encountered: