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

adding team members to 02-client and light-clients code owners #1563

Merged
merged 6 commits into from
Jun 23, 2022
9 changes: 7 additions & 2 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,15 @@
/modules/core/ @colin-axner @fedekunze @AdityaSripal
/proto/ibc/core/ @colin-axner @fedekunze @AdityaSripal

## CODEOWNERS for core/02-client

/modules/core/02-client @seantking @damiannolan
/proto/ibc/core/client @seantking @damiannolan
Comment on lines +19 to +22
Copy link
Member

Choose a reason for hiding this comment

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

Does /modules/core take a precedence over this or vice versa?
i.e. Should these lines include the existing codeowners?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was reading the code owners docs in GitHub and in theory the code owners in the higher level directory should have also still code ownership on the subdirectories... I can just add the full list of code owners here just in case, though.

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like this is not true. When I look at the 02-client directory, only @seantking and @damiannolan are code owners. You will need to add all code owners (later declarations take precedence over default owners)


# CODEOWNERS for the light-clients

/modules/light-clients/ @colin-axner @fedekunze @AdityaSripal
/proto/ibc/lightclients/ @colin-axner @fedekunze @AdityaSripal
/modules/light-clients/ @colin-axner @fedekunze @AdityaSripal @seantking @damiannolan
/proto/ibc/lightclients/ @colin-axner @fedekunze @AdityaSripal @seantking @damiannolan

# CODEOWNERS for ICS 20

Expand Down