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

Adjust readme for Ray Docs#204

Merged
Yard1 merged 3 commits intoray-project:masterfrom
Yard1:adjust_readme
Mar 29, 2022
Merged

Adjust readme for Ray Docs#204
Yard1 merged 3 commits intoray-project:masterfrom
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.

Let's try it without the table conversion, but otherwise great!

@Yard1 Yard1 merged commit b77d8da into ray-project:master Mar 29, 2022
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