Skip to content

[Test] Fix the lint from #29196#29975

Merged
jjyao merged 1 commit intoray-project:masterfrom
rkooo567:fix-lint-issue-event
Nov 3, 2022
Merged

[Test] Fix the lint from #29196#29975
jjyao merged 1 commit intoray-project:masterfrom
rkooo567:fix-lint-issue-event

Conversation

@rkooo567
Copy link
Contributor

@rkooo567 rkooo567 commented Nov 3, 2022

Signed-off-by: SangBin Cho rkooo567@gmail.com

Why are these changes needed?

Lint failed after it is merged due to some conflict. This should fix the issue

Related issue number

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 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
    • This PR is not tested :(

Signed-off-by: SangBin Cho <rkooo567@gmail.com>
@clarng clarng self-requested a review November 3, 2022 15:31
@jjyao jjyao merged commit 15f6a2d into ray-project:master Nov 3, 2022
WeichenXu123 pushed a commit to WeichenXu123/ray that referenced this pull request Dec 19, 2022
Lint failed after it is merged due to some conflict. This should fix the issue

Signed-off-by: SangBin Cho <rkooo567@gmail.com>
Signed-off-by: Weichen Xu <weichen.xu@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants