Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[lint] fix lint error #36789

Merged
merged 1 commit into from
Jun 24, 2023
Merged

[lint] fix lint error #36789

merged 1 commit into from
Jun 24, 2023

Conversation

aslonnie
Copy link
Collaborator

replace tab with space.

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
@aslonnie
Copy link
Collaborator Author

@aslonnie aslonnie assigned ericl and rickyyx and unassigned ericl Jun 24, 2023
@aslonnie aslonnie requested a review from rickyyx June 24, 2023 05:29
@aslonnie
Copy link
Collaborator Author

should we remove @wuisawesome and @DmitriGekhtman from code owner list?

@aslonnie
Copy link
Collaborator Author

not that I want code owners to be in the company, but I would like code owners to be responsive at least.

@pcmoritz pcmoritz merged commit 303df6a into master Jun 24, 2023
@pcmoritz pcmoritz deleted the lonnie-0623-fixlint branch June 24, 2023 08:48
@DmitriGekhtman
Copy link
Contributor

should we remove @wuisawesome and @DmitriGekhtman from code owner list?

I like getting updates on autoscaler-related PRs. (Though if you want to prune the list to people who actively maintain the code, I won't mind.)

You could add more code owners -- looks like @scv119 and @rickyyx are the main current contributors.

arvind-chandra pushed a commit to lmco/ray that referenced this pull request Aug 31, 2023
Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Signed-off-by: e428265 <arvind.chandramouli@lmco.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.

5 participants