Skip to content

Conversation

@qinxuye
Copy link
Collaborator

@qinxuye qinxuye commented May 13, 2022

What do these changes do?

This PR changed code owners to Mars PMC maintainers.

Related issue number

Fixes #xxxx

Check code requirements

  • tests added / passed (if needed)
  • Ensure all linting tests pass, see here for how to run them

@qinxuye qinxuye added this to the v0.10.0a1 milestone May 13, 2022
@qinxuye qinxuye requested review from hekaisheng and wjsi as code owners May 13, 2022 03:57
Copy link
Contributor

@hekaisheng hekaisheng left a comment

Choose a reason for hiding this comment

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

LGTM

@qinxuye
Copy link
Collaborator Author

qinxuye commented May 13, 2022

This PR did not touch code modification, thus skip CI.

Copy link
Contributor

@chaokunyang chaokunyang left a comment

Choose a reason for hiding this comment

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

LGTM

@wjsi
Copy link
Member

wjsi commented May 13, 2022

For specific modules, code owners can act as a mechanism for notification. Thus it might not be appropriate to remove all owners.

@wjsi wjsi merged commit be82602 into mars-project:master May 13, 2022
@qinxuye qinxuye deleted the codeowners branch May 13, 2022 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants