Skip to content

[Data] Rename Ray Data contributing guides and link to general Ray contributing guide#59320

Merged
bveeramani merged 3 commits intomasterfrom
address-review-comments2
Dec 10, 2025
Merged

[Data] Rename Ray Data contributing guides and link to general Ray contributing guide#59320
bveeramani merged 3 commits intomasterfrom
address-review-comments2

Conversation

@bveeramani
Copy link
Copy Markdown
Member

@bveeramani bveeramani commented Dec 9, 2025

This is a follow-up PR to address these review comments: #58589 (review).

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
@bveeramani bveeramani requested a review from a team as a code owner December 9, 2025 21:15
Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request correctly renames the Ray Data contributing guide and updates all necessary references. The new titles are more descriptive and improve the documentation's structure. The changes are clean and look good.

For a future improvement, consider updating the absolute URL on line 37 of doc/source/data/contributing/contributing-guide.md to a relative Sphinx reference (e.g., using the {ref} role). This would make the link more robust across different versions of the documentation. Since this line isn't in the current diff, this is just a suggestion for a follow-up.

@ray-gardener ray-gardener bot added docs An issue or change related to documentation data Ray Data-related issues labels Dec 10, 2025
Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
@bveeramani bveeramani enabled auto-merge (squash) December 10, 2025 04:19
@github-actions github-actions bot disabled auto-merge December 10, 2025 04:19
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Dec 10, 2025
@bveeramani bveeramani merged commit 4824f5a into master Dec 10, 2025
8 checks passed
@bveeramani bveeramani deleted the address-review-comments2 branch December 10, 2025 18:20
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
…ntributing guide (ray-project#59320)

This is a follow-up PR to address these review comments:
ray-project#58589 (review).

---------

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
Signed-off-by: peterxcli <peterxcli@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Ray Data-related issues docs An issue or change related to documentation go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants