Skip to content

update paramter name in save_csv#52541

Merged
bveeramani merged 2 commits intoray-project:masterfrom
iamjustinhsu:jhsu/fix-docs
Apr 23, 2025
Merged

update paramter name in save_csv#52541
bveeramani merged 2 commits intoray-project:masterfrom
iamjustinhsu:jhsu/fix-docs

Conversation

@iamjustinhsu
Copy link
Copy Markdown
Contributor

@iamjustinhsu iamjustinhsu commented Apr 22, 2025

Why are these changes needed?

  • docs don't match api
  • changing user guides to match api

Related issue number

Fixes #52534 (comment)

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • [ x ] This PR is not tested :(

@iamjustinhsu iamjustinhsu requested review from a team as code owners April 22, 2025 23:05
@bveeramani bveeramani self-assigned this Apr 22, 2025
@bveeramani bveeramani enabled auto-merge (squash) April 23, 2025 01:05
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Apr 23, 2025
@bveeramani bveeramani merged commit c935c62 into ray-project:master Apr 23, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-backlog go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Data][Docs] Code snippet in "Saving Data" doesn't work

3 participants