Skip to content
This repository was archived by the owner on Jan 12, 2026. It is now read-only.

Adjust readme for Ray Docs#30

Merged
Yard1 merged 6 commits intoray-project:mainfrom
Yard1:adjust_readme
Mar 29, 2022
Merged

Adjust readme for Ray Docs#30
Yard1 merged 6 commits intoray-project:mainfrom
Yard1:adjust_readme

Conversation

@Yard1
Copy link
Copy Markdown
Member

@Yard1 Yard1 commented Mar 25, 2022

Adjust the readme file to take advantage of ray-project/ray#23505

There are no user facing changes, the contents of the readme displayed on GitHub are the same as they were before.

Copy link
Copy Markdown
Collaborator

@krfricke krfricke left a comment

Choose a reason for hiding this comment

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

See xgboost

@Yard1 Yard1 merged commit f505c0f into ray-project:main Mar 29, 2022
@Yard1 Yard1 deleted the adjust_readme branch March 29, 2022 18:42
krfricke pushed a commit to ray-project/ray that referenced this pull request Mar 31, 2022
This PR fixes the issue of diverging documentation between Ray Docs and ecosystem library readmes which live in separate repos (eg. xgboost_ray). This is achieved by adding an extra step before the docs build process starts that downloads the readmes of specified ecosystem libraries from their GitHub repositories. The files are then preprocessed by a very simple parser to allow for differences between GitHub and Docs markdowns.

In summary, this makes the markdown files in ecosystem library repositories single sources of truth and removes the need to manually keep the doc pages up to date, all the while allowing for differences between what's rendered on GitHub and in the Docs.

See ray-project/xgboost_ray#204 & https://ray--23505.org.readthedocs.build/en/23505/ray-more-libs/xgboost-ray.html for an example.

Needs ray-project/xgboost_ray#204 and ray-project/lightgbm_ray#30 to be merged first.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants