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

Remove reviewers who have not joined the web-platform-tests org #11888

Merged
merged 1 commit into from
Jul 11, 2018

Conversation

foolip
Copy link
Member

@foolip foolip commented Jul 10, 2018

Part of #11293.

Reminder also in #11568.

This does not fully resolve #11293, as a few non-responders where
there would be no remaining reviewers are left, and one known to be
out-of-office. These will be handled separately to finish this.

Part of #11293.

Reminder also in #11568.

This does not fully resolve #11293, as a few non-responders where
there would be no remaining reviewers are left, and one known to be
out-of-office. These will be handled separately to finish this.
@foolip
Copy link
Member Author

foolip commented Jul 10, 2018

@youennf @thejohnjansen @rakuco, I've sent email about some folks being removed here if you want to go poking and not make this change.

@jgraham, care to review the change itself?

Copy link
Contributor

@marcoscaceres marcoscaceres left a comment

Choose a reason for hiding this comment

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

LGTM from payments side (@edenchuang will rejoin next month, so we can add him back then).

Same on the manifest side.

Copy link
Contributor

@sideshowbarker sideshowbarker left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@rakuco rakuco left a comment

Choose a reason for hiding this comment

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

Intel lgtm

@foolip
Copy link
Member Author

foolip commented Jul 11, 2018

Thanks all, I'll go ahead and merge.

To anyone who was removed here and finds out later, fear not, you can ask here or on IRC to get another invite to the web-platform-tests org and get yourself added back.

@foolip foolip merged commit 8fea122 into master Jul 11, 2018
@foolip foolip deleted the remove-suggested-reviewers branch July 11, 2018 09:04
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.

Remove OWNERS who have not joined the web-platform-tests org
5 participants