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
[BEAM-4790] Add Prow-style OWNERS files. #6235
Conversation
Thanks @udim I can’t add to this branch. Can you please add me on the new io/kudu? Thank you |
Based on the last CODEOWNERS version, removed in https://github.com/apache/beam/pull/6049/files.
@timrobertson100 done! |
Thanks! This LGTM. |
@@ -0,0 +1,4 @@ | |||
# See the OWNERS docs at https://s.apache.org/beam-owners |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This URL doesn't resolve for me: https://s.apache.org/beam-owners
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was supposed to link to https://beam.apache.org/contribute/#reviews .
I forgot to ask someone with permissions to set that. I'd appreciate it if you could set it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Things to do (in another PR):
Follow this checklist to help us incorporate your contribution quickly and easily:
[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.It will help us expedite review of your Pull Request if you tag someone (e.g.
@username
) to look at it.Post-Commit Tests Status (on master branch)